The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #15036

Race condition to search when autorefresh is enabled

Added by Jo Rhett almost 2 years ago. Updated about 1 year ago.

Status:Merged - Pending ReleaseStart date:06/14/2012
Priority:NormalDue date:
Assignee:Aaron Stone% Done:

0%

Category:-
Target version:-
Keywords: Affected URL:
Branch: Affected Dashboard version:

Description

If you enable autorefresh and then click on File Search in the top toolbar, it will refresh every few seconds and wipe out the text you are trying to type into the input fields. This requires the user to disable autorefresh to use this form, then enable it again when they leave.

I think you should disable autorefresh on this page, and re-enable it when they leave. Or ignore it on this page. Allow the page to work, without requiring the user to hack their way around it.

History

#1 Updated by Aaron Stone about 1 year ago

  • Status changed from Unreviewed to Accepted
  • Assignee set to Aaron Stone

In the Rails 3 branch, I have replaced the autorefresh code with an entirely client-side implementation that can be more easily be context-sensitive to form pages. (However my new code will replicate this bug faithfully at the moment.)

#2 Updated by Aaron Stone about 1 year ago

  • Target version deleted (1.2.x)

#3 Updated by Marko Locher about 1 year ago

This problem not only affects the search form, but any other formular on the dashboard. Run into it while creating groups on version 1.2.23

#4 Updated by Aaron Stone about 1 year ago

  • Status changed from Accepted to In Topic Branch Pending Review

Code fix for rails3 branch. It’s not feasible to backport this to 1.2, sorry! https://github.com/puppetlabs/puppet-dashboard/pull/210

#5 Updated by Aaron Stone about 1 year ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

Fixed for rails3 branch.

Also available in: Atom PDF