Exclude "/" from pipeline names too as it causes problems in URLs
Created by: muffato
Use case
Jorge found this one by accident. He had put a /
in the pipeline_name
option and then generate_graph.pl failed with this error
Error: <stdin>: syntax error in line 119 near '/'
When doing my own tests, I realized that the URLs themselves would be confusing with a /
in the database name, so better to remove them early on.
Description
Just like whitespace, replace the /
with an underscore
Possible Drawbacks
N/A
Testing
Have you added/modified unit tests to test the changes?
No, that function is not currently tested.
If so, do the tests pass/fail?
Have you run the entire test suite and no regression was detected?