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

Fix identical status label in simplified dashboard #3455

Open
wants to merge 8 commits into
base: support/2.13.0
Choose a base branch
from

Conversation

RomainLvr
Copy link

@RomainLvr RomainLvr commented Feb 9, 2024

Changes description

Update the STATUS_WAITING label to avoid an identical status label with PENDING. This confusion appears in My requests for assistance (Simplified dashboard) when GLPI is in French for example.

Before :
Capture d’écran du 2024-02-09 14-19-33

After :
Capture d’écran du 2024-02-09 14-20-09

Checklist

Please check if your PR fulfills the following specifications:

  • Tests for the changes have been added
  • Docs have been added/updated

References

Ticket !31528

@RomainLvr RomainLvr self-assigned this Feb 9, 2024
@RomainLvr RomainLvr requested a review from Rom1-B February 9, 2024 14:24
@@ -75,7 +75,7 @@ class PluginFormcreatorFormAnswer extends CommonDBTM

public static function getStatuses() {
return [
self::STATUS_WAITING => __('Waiting', 'formcreator'),
self::STATUS_WAITING => __('Waiting for approval'),
Copy link
Contributor

Choose a reason for hiding this comment

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

In this case, the variable name is ambiguous

Copy link
Author

Choose a reason for hiding this comment

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

Fixed in d53bd46

inc/formanswer.class.php Outdated Show resolved Hide resolved
inc/formanswer.class.php Outdated Show resolved Hide resolved
inc/issue.class.php Outdated Show resolved Hide resolved
inc/issue.class.php Show resolved Hide resolved
Comment on lines +60 to 61
const STATUS_APPROVAL = 101;
const STATUS_WAITING = 101;
Copy link
Contributor

Choose a reason for hiding this comment

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

101 is used twice

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

Successfully merging this pull request may close these issues.

None yet

3 participants