search

redaxo / redaxo4

REDAXO 4
https://redaxo.org
41 stars 26 forks source link

rex_list fix #423

Closed bloep closed 6 years ago

bloep commented 6 years ago

Refs https://github.com/redaxo/redaxo/issues/2043

gharlan commented 6 years ago

Passt der Fix hier wirklich? In R5 nutzen wir escapeIdentifier, was es in R4 noch nicht gab. Hier nutzen wir escape, was im Gegensatz zu R5 keine Anführungszeichen drum rum setzt, und eigentlich ja auch nicht für Identifier gedacht ist, sondern für Werte.

Keine Ahnung, vielleicht kommt es letztlich trotzdem hin, ich fürchte aber, wenn der Spaltenname Sonderzeichen enthält, dass es dann hier ein Problem gibt.

staabm commented 5 years ago

Ich befürchte gregor hat recht. Vllt sollten wir aus R5 escapdeIdentifier nach R4 portieren

staabm commented 5 years ago

Ich befürchte gregor hat recht. Vllt sollten wir aus R5 escapdeIdentifier nach R4 portieren

umgesetzt in https://github.com/redaxo/redaxo4/pull/430