Project

General

Profile

Actions

Bug #19228

closed

Date picker in search (last inventory) is broken

Added by François ARMAND almost 3 years ago. Updated almost 3 years ago.

Status:
Released
Priority:
N/A
Category:
Web - UI & UX
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Very Small
Priority:
79
Name check:
To do
Fix check:
Checked
Regression:

Description

If you put a date in ISO format, you get an error, and the error doesn't even tell you what format is expected (see screenshot).
There should be a datepicker to relieve us from the guessing game, but it is broken:

Uncaught TypeError: init is undefined
    <anonymous> https://orchestrateur-4.labo.normation.com/rudder/secure/nodeManager/searchNodes#{"query":{"select":"node","composition":"And","where":[{"objectType":"node","attribute":"OS","comparator":"eq","value":"Linux"}]}} line 2 > injectedScript:5
    jQuery 9
    <anonymous> https://orchestrateur-4.labo.normation.com/rudder/secure/nodeManager/searchNodes#{"query":{"select":"node","composition":"And","where":[{"objectType":"node","attribute":"OS","comparator":"eq","value":"Linux"}]}} line 2 > injectedScript:4
    setTimeout handler* https://orchestrateur-4.labo.normation.com/rudder/secure/nodeManager/searchNodes#{"query":{"select":"node","composition":"And","where":[{"objectType":"node","attribute":"OS","comparator":"eq","value":"Linux"}]}} line 2 > injectedScript:2
    jQuery 8
    ajaxPost https://orchestrateur-4.labo.normation.com/rudder/classpath/lift.js:702
    doAjaxCycle https://orchestrateur-4.labo.normation.com/rudder/classpath/lift.js:280
    appendToQueue https://orchestrateur-4.labo.normation.com/rudder/classpath/lift.js:138
    onchange https://orchestrateur-4.labo.normation.com/rudder/secure/nodeManager/searchNodes#{"query":{"select":"node","composition":"And","where":[{"objectType":"node","attribute":"OS","comparator":"eq","value":"Linux"}]}}:1

We need to:
- accept date/time in iso format,
- tell the problem in the error message,
- repair the datepicker

All that without breaking existing date stored (so perhaps the iso date format will have to wait 7.0).

Actions #1

Updated by François ARMAND almost 3 years ago

  • Status changed from New to In progress
Actions #2

Updated by François ARMAND almost 3 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Raphael GAUTHIER
  • Pull Request set to https://github.com/Normation/rudder/pull/3625
Actions #3

Updated by François ARMAND almost 3 years ago

  • Status changed from Pending technical review to Pending release
Actions #4

Updated by François ARMAND almost 3 years ago

  • Subject changed from Date format in search is French localised and the datepicker is broken to Date picker in search (last inventory) is broken
Actions #5

Updated by Nicolas CHARLES almost 3 years ago

  • Fix check changed from To do to Checked
Actions #6

Updated by Vincent MEMBRÉ almost 3 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.13 and 6.2.7 which were released today.

Actions

Also available in: Atom PDF