Paths

Table of Contentst

Differential D15551

Converting badge quality property from color to an integer representation for later sorting purposes
ClosedPublic
Actions

Authored by lpriestley on Mar 31 2016, 12:11 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 7, 11:25 AM

	Unknown Object (File)
	Sun, Nov 24, 2:28 PM

	Unknown Object (File)
	Thu, Nov 21, 7:38 PM

	Unknown Object (File)
	Nov 17 2024, 5:28 PM

	Unknown Object (File)
	Nov 16 2024, 2:59 PM

	Unknown Object (File)
	Nov 14 2024, 9:55 AM

	Unknown Object (File)
	Nov 9 2024, 7:40 PM

	Unknown Object (File)
	Nov 6 2024, 6:02 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T9007: Allow Badges to be ordered by quality

Commits

rP00425cac94d5: Converting badge quality property from color to an integer representation for…

Summary

Test Plan

Create badges, update quality, search by quality without change of functionality.

Diff Detail

Repository

Branch

badgesquality

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 11357
Build 14136: Run Core Tests
Build 14135: arc lint + arc unit

Event Timeline

lpriestley updated this revision to Diff 37492.Mar 31 2016, 12:11 AM

lpriestley retitled this revision from to Converting badge quality property from color to an integer representation for later sorting purposes.

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T9007: Allow Badges to be ordered by quality.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMar 31 2016, 12:11 AM

Herald added a subscriber: Korvin. · View Herald Transcript

Some minor inline stuff but this looks broadly correct to me.

src/applications/badges/constants/PhabricatorBadgesQuality.php
15–17	Weird indent
20–24	Can this just be `return $quality`? Do we need this at all? I don't think anything calls it, and it can just be deleted.
src/applications/badges/editor/PhabricatorBadgesEditEngine.php
90–91	I think this should just be `$object->getQuality()` -- the map will be a map from `100 => "Uncommon"` but the "value" of a select control is the key (100), not the string ("Uncommon"). I would expect this to cause a bug here: Set a badge to a quality like "Legendary". Save it. Click "Edit Badge". The <select /> dropdown is now set to the wrong value (not "Legendary").
src/applications/badges/storage/PhabricatorBadgesBadge.php
28	Probably move this to `PhabricatorBadgesQuality`?

This revision is now accepted and ready to land.Mar 31 2016, 12:20 AM

Moving things around

Closed by commit rP00425cac94d5: Converting badge quality property from color to an integer representation for… (authored by lpriestley, committed by lpriestley). · Explain WhyMar 31 2016, 12:28 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20160330.badges.migratequality.sql

32 lines

20160330.badges.qualityxaction.mig.sql

59 lines

src/

__phutil_library_map__.php

2 lines

applications/

badges/

constants/

PhabricatorBadgesQuality.php

82 lines

controller/

PhabricatorBadgesViewController.php

5 lines

editor/

PhabricatorBadgesEditEngine.php

5 lines

PhabricatorBadgesEditor.php

3 lines

query/

PhabricatorBadgesSearchEngine.php

9 lines

storage/

PhabricatorBadgesBadge.php

24 lines

PhabricatorBadgesTransaction.php

5 lines

view/

phui/

PHUIBadgeMiniView.php

4 lines

PHUIBadgeView.php

43 lines

Diff 37492

resources/sql/autopatches/20160330.badges.migratequality.sql

Loading...

resources/sql/autopatches/20160330.badges.qualityxaction.mig.sql

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/badges/constants/PhabricatorBadgesQuality.php

Loading...

src/applications/badges/controller/PhabricatorBadgesViewController.php

Loading...

src/applications/badges/editor/PhabricatorBadgesEditEngine.php

Loading...

src/applications/badges/editor/PhabricatorBadgesEditor.php

Loading...

src/applications/badges/query/PhabricatorBadgesSearchEngine.php

Loading...

src/applications/badges/storage/PhabricatorBadgesBadge.php

Loading...

src/applications/badges/storage/PhabricatorBadgesTransaction.php

Loading...

src/view/phui/PHUIBadgeMiniView.php

Loading...

src/view/phui/PHUIBadgeView.php

Loading...