Project

General

Profile

Actions
Copy link

Bug #15509

closed

Prevent setting an empty report component

Added by Vincent MEMBRÉ over 4 years ago. Updated almost 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - Technique editor
Target version:
5.0.15
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Very Small
Priority:
51
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

We can set an empty string as report component in the technique editor which is REALLY confusing. Setting an empty string should be used to reset to default value (name of the method, which is a better way to achieve that than setting method_call )

A fix shuld be made on 5.0 and another one in 5.1

Subtasks 1 (0 open — 1 closed)

Bug #16226: Error when importing technique with missing component in a method callReleasedFrançois ARMANDActions
Actions
Copy link
 #1

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions
Copy link
 #2

Updated by François ARMAND over 4 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Effort required set to Very Small
  • Priority changed from 0 to 61
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
  • Priority changed from 61 to 59
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
  • Pull Request set to https://github.com/Normation/ncf/pull/1104
Actions
Copy link
 #6

Updated by François ARMAND over 4 years ago

  • Assignee changed from François ARMAND to Vincent MEMBRÉ
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:9b26466f08235949040a11040d220fca2e577730.

Actions
Copy link
 #8

Updated by Alexis Mousset over 4 years ago

  • Name check changed from To do to Reviewed
Actions
Copy link
 #9

Updated by Alexis Mousset over 4 years ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #10

Updated by Vincent MEMBRÉ over 4 years ago

This bug has been fixed in Rudder 5.0.15 which was released today.

Actions
Copy link
 #11

Updated by Vincent MEMBRÉ over 3 years ago

  • Status changed from Pending release to Released
  • Priority changed from 59 to 51
Actions
Copy link
 #12

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
Actions
Copy link

Also available in: Atom PDF