Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix class selection in update #1142

Merged
merged 3 commits into from
Jun 29, 2020

Conversation

sebsoftware
Copy link
Contributor

@sebsoftware sebsoftware commented Jun 28, 2020

This fixes an issue with the class selector in an update expression.

i.E.

update="mainInfo formBeaninfo @(.message)"

was working well, resolving newUpdate to

"fEditor:mainInfo fEditor:formBeaninfo fEditor:j_idt84"

but an

update="mainInfo @(.message) formBeaninfo"

was evaluated to wrong concatenation:

"fEditor:mainInfo fEditor:j_idt84fEditor:formBeaninfo "

This fixes an issue with the class selector in an update expression.

i.E.

update="mainInfo formBeaninfo @(.message)"

was working well, resolving newUpdate to

"fEditor:mainInfo fEditor:formBeaninfo fEditor:j_idt84"

but an

update="mainInfo @(.message) formBeaninfo"

was evaluated to wrong concatenation:

"fEditor:mainInfo  fEditor:j_idt84fEditor:formBeaninfo "
@geopossachs geopossachs self-assigned this Jun 28, 2020
@stephanrauh stephanrauh merged commit 18f1d13 into TheCoder4eu:master Jun 29, 2020
@stephanrauh
Copy link
Collaborator

Looks fine by me. Thanks, Jens!

Best regards,
Stephan

@stephanrauh stephanrauh added bug confirmed Solved This ticket has been solved. If it's still open, we're waiting for the user's confirmation. labels Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug confirmed Solved This ticket has been solved. If it's still open, we're waiting for the user's confirmation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants