Page MenuHomePhabricator

Fixed inconsistency at phriction

Authored by Luke081515.2 on Sep 10 2016, 10:15 PM.
Referenced Files
Unknown Object (File)
Tue, Nov 28, 3:15 AM
Unknown Object (File)
Tue, Nov 28, 3:15 AM
Unknown Object (File)
Sat, Nov 25, 5:47 AM
Unknown Object (File)
Thu, Nov 23, 5:32 AM
Unknown Object (File)
Mon, Nov 13, 12:52 PM
Unknown Object (File)
Wed, Nov 8, 4:05 AM
Unknown Object (File)
Mon, Nov 6, 7:10 AM
Unknown Object (File)
Oct 29 2023, 9:17 PM


  • All Applications are useing the description "Tags", Phriction actually uses "Projects"
  • Changed "Projects" to "Tags" for phriction

Ref T11614

Test Plan
  • Edited a document, took a look at the description for that input line
  • Added a project to a page and saved it

Diff Detail

rP Phabricator
Lint Errors
Errorsrc/applications/auth/provider/PhabricatorSlackAuthProvider.php:29PHL1Unknown Symbol
Tests Passed
Build Status
Buildable 13677
Build 17638: arc lint + arc unit

Event Timeline

Luke081515.2 retitled this revision from to Fixed inconsistency at phriction.
Luke081515.2 updated this object.
Luke081515.2 edited the test plan for this revision. (Show Details)
Luke081515.2 added a project: Phriction.

Did you try to actually set a project tag with this change? I'm pretty sure it wouldn't work.

chad requested changes to this revision.Sep 11 2016, 12:36 AM
chad added a reviewer: chad.

This shouldn't work, since your changing the variable name as well below, but not above to write the changes. I'd expect this to not record the tag.

This revision now requires changes to proceed.Sep 11 2016, 12:36 AM

At least it did what I wanted, and it worked without an error:

before.PNG (551×288 px, 9 KB)

after.PNG (492×236 px, 6 KB)

Did you click "Submit" and verify the tag you specified was on the document?

hm, ok, that was the reason. I don't checked that, sorry. Feel free to comandeer that rev, and upload a better patch.


just change setName back to projects and it should work again. this field isn't visible to users.

Luke081515.2 edited edge metadata.
  • Fixed issue pointed out by @chad
Luke081515.2 edited edge metadata.
Luke081515.2 marked an inline comment as done.
chad edited edge metadata.


This revision is now accepted and ready to land.Sep 11 2016, 6:06 PM

@chad Can you land this revision yourself, or give me the permissions to do so? I'm currently not able too :-/

This revision was automatically updated to reflect the committed changes.