Skip to content

Conversation

@eliranshlomo
Copy link

This change adds a generateJobPath() helper function which turns job
names into encoded job names. Job names are split on '/', each part is
run through encodeValue(), then the resulting encoded parts are joined
by "/job/". All instances where encodeValue(job) was used to build a job
URL have been replaced with calls to generateJobPath().

A new test was added to handle the case where a job is in a folder.

This change adds a generateJobPath() helper function which turns job
names into encoded job names. Job names are split on '/', each part is
run through encodeValue(), then the resulting encoded parts are joined
by "/job/". All instances where encodeValue(job) was used to build a job
URL have been replaced with calls to generateJobPath().

A new test was added to handle the case where a job is in a folder.
@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@saxonww
Copy link

saxonww commented Nov 16, 2015

@eliranshlomo, this appears to be identical to #16, including the commit message. Is this what you intended?

@eliranshlomo
Copy link
Author

No, pushed the wrong button.
sorry.

@cashlalala cashlalala closed this May 11, 2018
cashlalala pushed a commit that referenced this pull request May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants