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

Modify end_tag_regex to allow named arrays #144

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

michaelalandawson
Copy link
Contributor

When determining if a custom tag uses an end tag, the regex should also allow for the following:

thistag["executionmode"]
thistag['executionmode']

Currently, the regex only allows for thistag.executionmode to determine the end tag.

The proposed regex was tested with the following variations:

thistag.executionmode == "end"
thistag.executionmode == 'end'
thistag.executionmode eq "end"
thistag.executionmode eq 'end'
thistag.executionmode is "end"
thistag.executionmode is 'end'
thistag["executionmode"] == "end"
thistag["executionmode"] == 'end'
thistag["executionmode"] eq "end"
thistag["executionmode"] eq 'end'
thistag["executionmode"] is "end"
thistag["executionmode"] is 'end'
thistag['executionmode'] == "end"
thistag['executionmode'] == 'end'
thistag['executionmode'] eq "end"
thistag['executionmode'] eq 'end'
thistag['executionmode'] is "end"
thistag['executionmode'] is 'end'

Thank you!

Michael Dawson and others added 5 commits December 8, 2021 11:57
- add CF2021 array member functions
- change case for consistency: "metaData" -> "metadata"
When determining if a custom tag uses an end tag, the regex should also allow for the following:

thistag["executionmode"]
thistag['executionmode']

Currently, the regex only allows for thistag.executionmode to determine the end tag.
…g_regex

Modify end_tag_regex to allow named arrays
@michaelalandawson michaelalandawson force-pushed the master branch 2 times, most recently from 9e816f7 to a3a35e6 Compare October 9, 2023 01:46
- Allow for the following variable syntax variations:
  attributes.var
  attributes["var"]
  attributes['var']
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant