Project

General

Profile

Actions

User story #6368

closed

Methods named check_* should have an equivalent without reporting

Added by Benoît PECCATTE about 9 years ago. Updated 8 months ago.

Status:
Resolved
Priority:
N/A
Assignee:
-
Category:
Generic methods
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:
No

Description

Methods such as file_check_exists report on their result whereas we may not want to.
There are 2 use cases :
- we want the file to be present and we want the method to tell us if not
- we want to know wether the file is present or not and take an action based on this

The second case is a new methods.
We could name them does_* instead of check_*, for example file_does_exist


Related issues 1 (0 open1 closed)

Has duplicate Rudder - Architecture #8683: check_* methods output error messages when the result of a check is "no"ResolvedActions
Actions #1

Updated by François ARMAND almost 8 years ago

  • Has duplicate Architecture #8683: check_* methods output error messages when the result of a check is "no" added
Actions #2

Updated by Alexis Mousset over 7 years ago

  • Category set to Generic methods
Actions #3

Updated by Alexis Mousset about 7 years ago

  • Target version changed from 0.x to master
Actions #4

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from master to 4.2.2
Actions #5

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.2.2 to 4.2.3
Actions #6

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.2.3 to 4.2.4
Actions #7

Updated by Benoît PECCATTE about 6 years ago

  • Target version changed from 4.2.4 to Ideas (not version specific)
Actions #8

Updated by Benoît PECCATTE over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Generic methods to Generic methods
Actions #9

Updated by Alexis Mousset 8 months ago

  • Status changed from New to Resolved
  • Regression set to No

we can now disable reporting

Actions

Also available in: Atom PDF