Merge pull request #153 from joehoyle/add-name-function-macro

TobiasBengtsson · web-flow · commit 73902ef017e1 · 2022-09-30T13:04:22.000+02:00
Add ability to set function name on php_function macro
diff --git a/crates/macros/src/function.rs b/crates/macros/src/function.rs
@@ -17,6 +17,7 @@ pub struct AttrArgs {
     optional: Option<String>,
     ignore_module: bool,
     defaults: HashMap<String, Lit>,
+    name: Option<String>,
 }
 
 #[derive(Debug, Clone)]
@@ -93,7 +94,7 @@ pub fn parser(args: AttributeArgs, input: ItemFn) -> Result<(TokenStream, Functi
     }
 
     let function = Function {
-        name: ident.to_string(),
+        name: attr_args.name.unwrap_or_else(|| ident.to_string()),
         docs: get_docs(&input.attrs),
         ident: internal_ident.to_string(),
         args,

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ pub struct AttrArgs {`
`17`	`17`	`optional: Option<String>,`
`18`	`18`	`ignore_module: bool,`
`19`	`19`	`defaults: HashMap<String, Lit>,`
	`20`	`+ name: Option<String>,`
`20`	`21`	`}`
`21`	`22`
`22`	`23`	`#[derive(Debug, Clone)]`
`@@ -93,7 +94,7 @@ pub fn parser(args: AttributeArgs, input: ItemFn) -> Result<(TokenStream, Functi`
`93`	`94`	`}`
`94`	`95`
`95`	`96`	`let function = Function {`
`96`		`- name: ident.to_string(),`
	`97`	`+ name: attr_args.name.unwrap_or_else(\|\| ident.to_string()),`
`97`	`98`	`docs: get_docs(&input.attrs),`
`98`	`99`	`ident: internal_ident.to_string(),`
`99`	`100`	`args,`