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

[JENKINS-72358] Replace Prototype.js $(var) with var #101

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

MarkEWaite
Copy link
Contributor

[JENKINS-72358] Replace Prototype.js $(var) with var

JENKINS-72358 reports a JavaScript error when the plus sign is clicked to expand the test results analyzer tree.

The blog post recommends replacing $(variable) with variable. That's what I did in 3 cases in this change.

I've confirmed that the JavaScript message that was displayed with the released version of the plugin is not displayed with these changes.

Testing done

Interactively verified that I could see the JavaScript warning messages in the web developer console before this change and that the warning messages did not appear after the change. The UI behaved incorrectly before the change and behaved correctly after the change.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

https://issues.jenkins.io/browse/JENKINS-72358 reports a JavaScript error
when the plus sign is clicked to expand the test results analyzer tree.

https://www.jenkins.io/blog/2023/05/12/removing-prototype-from-jenkins/
recommends replacing `$(variable)` with `variable`.  That's what I did in
3 cases in this change.

I've confirmed that the JavaScript message that was displayed with the
released version of the plugin is not displayed with these changes.
@MarkEWaite MarkEWaite added the bug label Nov 22, 2023
@MarkEWaite
Copy link
Contributor Author

MarkEWaite commented Nov 22, 2023

@gounthar you are currently a maintainer of this plugin. Would you be willing to review and merge this pull request?

Copy link
Contributor

@gounthar gounthar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, Mark. It looks good to me, at least with my understanding of JS.

@gounthar gounthar merged commit 99feabb into jenkinsci:master Nov 22, 2023
14 checks passed
@MarkEWaite MarkEWaite deleted the replace-prototype-js branch November 22, 2023 16:28
@Pexkoh
Copy link

Pexkoh commented Dec 6, 2023

Hi @gounthar,
Thank you for merging Mark’s pull request – it fixed the problem we had with the plugin on our Jenkins server.

I’m pretty new to Jenkins so I hope you don’t mind me asking – is it planned to add the current fix to the official Jenkins plugin releases (if you have the necessary access to change the .hpi file)?
This way we could continue using the official plugin and still receive potential updates if someone decides to adopt and extend it.

I appreciate the help!

@gounthar
Copy link
Contributor

gounthar commented Dec 7, 2023

Let's see if Mark and I can publish a release today. 🤞

@gounthar
Copy link
Contributor

gounthar commented Dec 7, 2023

I’m pretty new to Jenkins so I hope you don’t mind me asking – is it planned to add the current fix to the official Jenkins plugin releases (if you have the necessary access to change the .hpi file)? This way we could continue using the official plugin and still receive potential updates if someone decides to adopt and extend it.

Absolutely, it's perfectly normal and welcome to ask for a release when a fix you're looking for gets merged. Your input is valuable, and we appreciate your interest in using Jenkins and this plugin. 🚀

Feel free to reach out if you have any more questions or if there's anything else you'd like to discuss.

@gounthar
Copy link
Contributor

gounthar commented Dec 7, 2023

The release 0.41.0 is available, thanks to the joint efforts of Mark and myself. 🚀

@Pexkoh
Copy link

Pexkoh commented Dec 8, 2023

Thanks a lot for the released version and the lightning fast reactions 😉 I really appreciate the efforts from both of you and the kind words!

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

Successfully merging this pull request may close these issues.

3 participants