https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26635 --- Comment #100 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Tomás Cohen Arazi from comment #99)
(In reply to Jonathan Druart from comment #98)
1. "Add optional support for `:` as the modifier" We should document that. Why do we have it here actually? Shouldn't it be on its own bug?
I agree.
2. Cannot we come with something better than "_str" and "str"?
I agree with that, you proposed. You don't like?
Yes, the idea, but I was expecting someone to tell me that "str" is ugly (especially when appearing twice :D) The second one can be 'str', not the first one.
3. We still have specific reference to "av" when I think we should focus on "stringify coded values" as a general concept.
AV expansion can be expensive. And so other strings. The idea of ':', was to later add <attribute>:str, so things that are not linked to av, might be generated on demand as well.
In my mind consumers don't want to ask for each attributes to be expanded, but one "expand everything" (actually "stringify everything") switch. I may be wrong. Maybe we are mixing "expand" and "stringify" into the same concept and we should not. -- You are receiving this mail because: You are watching all bug changes.