Ecto allows parentheses in field names without raising an error

[dkuku]

Elixir version

latest

Database and Version

any?

Ecto Versions

main

Database Adapter and Versions (postgrex, myxql, etc)

ecto without adapters

Current behavior

Adding parentheses to field names (e.g., q.title()) is accepted and processed as if the parentheses weren't present. This behavior could be misleading.

Example:

# This works:
order_by("q", [q], [asc: q.title()])

# Same as:
order_by("q", [q], [asc: q.title])

Expected behavior

I would expect to get an error

[josevalim]

We should definitely warn and then convert to an error in the future.

[greg-rychlewski]

@josevalim Is the only way to differentiate them in the AST by no_parens: true in the metadata?

[josevalim]

Yes!