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

Creating a new workflow definition without title throws 500 #420

Open
shoosah opened this issue Apr 7, 2020 · 2 comments
Open

Creating a new workflow definition without title throws 500 #420

shoosah opened this issue Apr 7, 2020 · 2 comments

Comments

@shoosah
Copy link
Contributor

shoosah commented Apr 7, 2020

I suspect this is related to the cases when the existing titles contain parentheses like Content Approver Group: Subject group 10 (Media Studies,Technology). In which case, the following happens during the check for existing titles:
[Warning] preg_match(): Compilation failed: missing closing parenthesis at offset 47
preg_match(#Content Approver Group: Subject group 10 (Media#, My Workflow) WorkflowDefinition.php:437

@robbieaverill
Copy link
Contributor

Sounds like we need to preg_quote() the input before this line. Would you like to make a PR if that works for you?

@shoosah
Copy link
Contributor Author

shoosah commented Apr 7, 2020

Sounds like we need to preg_quote() the input before this line. Would you like to make a PR if that works for you?

Here is the PR: #421

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants