https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38342

--- Comment #2 from Tomás Cohen Arazi (tcohen) <[email protected]> ---
@ashimema: I have doubts about the handling you added for the 'property' value.
Specifically:

```
   property => $property =~ /(\w+\.\w+)$/
         ? $1
         : $property,    # results in table.column without quotes or backtics
```

Before that check you added for cleaning it up, there's a use of `$property`
as-is for checking the column is of type ENUM. So I feel either my patch is
correct, or we might miss some cases for enum types (i.e. the cleanup needs to
happen for all uses of `$property`, not just the last one).

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to