Extract function positional arguments as Fluent entries placeables

CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## Unreleased - [0.1.11]
 
+### Bug fixes
+
+- Fix translations checker not extracting Fluent function positional arguments
+  as placeables.
+
 ### Enhancements
 
 - Accept multiple configuration conditional checks for the same parameters in
@@ -423,7 +428,7 @@ version to `0.1` during installation.
 
 - Added all ISO-639-1 and ISO-639-2 languages.
 
-[0.1.11]: https://github.com/mondeja/leptos-fluent/compare/v0.1.10...master
+[0.1.11]: https://github.com/mondeja/leptos-fluent/compare/v0.1.10...v0.1.11
 [0.1.10]: https://github.com/mondeja/leptos-fluent/compare/v0.1.9...v0.1.10
 [0.1.9]: https://github.com/mondeja/leptos-fluent/compare/v0.1.8...v0.1.9
 [0.1.8]: https://github.com/mondeja/leptos-fluent/compare/v0.1.7...v0.1.8

leptos-fluent-macros/src/translations_checker/fluent_entries.rs

@@ -18,13 +18,32 @@ fn get_fluent_entries_from_resource(
                 let mut placeables = Vec::new();
                 for element in &value.elements {
                     if let fluent_syntax::ast::PatternElement::Placeable {
-                        expression: fluent_syntax::ast::Expression::Inline(
+                        expression,
+                    } = element
+                    {
+                        if let fluent_syntax::ast::Expression::Inline(
                             fluent_syntax::ast::InlineExpression::VariableReference {
                                 id
                             }
-                        )
-                    } = element {
-                        placeables.push(id.name.to_string());
+                        ) = expression {
+                            placeables.push(id.name.to_string());
+                        } else if let fluent_syntax::ast::Expression::Inline(
+                            fluent_syntax::ast::InlineExpression::FunctionReference {
+                                arguments: fluent_syntax::ast::CallArguments {
+                                    positional,
+                                    ..
+                                },
+                                ..
+                            }
+                        ) = expression {
+                            for arg in positional {
+                                if let fluent_syntax::ast::InlineExpression::VariableReference {
+                                    id
+                                } = arg {
+                                    placeables.push(id.name.to_string());
+                                }
+                            }
+                        }
                     }
                 }
                 entries.push(FluentEntry {
@@ -322,4 +341,50 @@ mod tests {
             )])
         );
     }
+
+    #[test]
+    fn test_fluent_functions() {
+        let fluent_resources = HashMap::from([(
+            "en-US".to_string(),
+            vec![
+                r#"locale-date-format = { DATETIME($date, month: "long", year: "numeric", day: "numeric") }
+log-time2 = Entry time: { DATETIME($date) }
+emails2 = Number of unread emails { NUMBER($unreadEmails) }
+"#.to_string()
+            ],
+        )]);
+        let fluent_file_paths = HashMap::from([(
+            "en-US".to_string(),
+            vec!["./locales/en-US/foo.ftl".to_string()],
+        )]);
+        let workspace_path = "./";
+        let (entries, errors) = build_fluent_entries(
+            &fluent_resources,
+            &fluent_file_paths,
+            workspace_path,
+            &None,
+            &None,
+        );
+        assert!(errors.is_empty());
+        assert_eq!(
+            entries,
+            HashMap::from([(
+                "en-US".to_string(),
+                vec![
+                    FluentEntry {
+                        message_name: "locale-date-format".to_string(),
+                        placeables: vec!["date".to_string()]
+                    },
+                    FluentEntry {
+                        message_name: "log-time2".to_string(),
+                        placeables: vec!["date".to_string()]
+                    },
+                    FluentEntry {
+                        message_name: "emails2".to_string(),
+                        placeables: vec!["unreadEmails".to_string()]
+                    }
+                ]
+            )])
+        );
+    }
 }