Page MenuHomePhabricator

Use have/need data in ArcanistPhutilLibraryLinter
ClosedPublic

Authored by epriestley on Jul 16 2014, 10:58 PM.
Tags
None
Referenced Files
F15421474: D9954.id23888.diff
Sat, Mar 22, 12:06 AM
F15397812: D9954.diff
Sun, Mar 16, 10:24 PM
F15393031: D9954.diff
Sat, Mar 15, 7:02 PM
F15390791: D9954.id23914.diff
Sat, Mar 15, 6:40 AM
F15381710: D9954.diff
Fri, Mar 14, 9:01 AM
F15345001: D9954.id23888.diff
Mon, Mar 10, 8:03 AM
Unknown Object (File)
Sun, Mar 2, 6:13 PM
Unknown Object (File)
Feb 9 2025, 2:16 PM
Subscribers

Details

Summary

Ref T5640. In D9864, the data this linter pulls out of the map was changed, breaking "use of undeclared function" warnings.

Test Plan
>>> Lint for src/future/FutureProxy.php:


   Error  (PHL1) Unknown Symbol
    Use of unknown function 'qqqqqq'. Common causes are:
    
      - Your libphutil/ is out of date.
        This is the most common cause.
        Update this copy of libphutil: /INSECURE/devtools/libphutil
    
      - Some other library is out of date.
        Update the library this symbol appears in.
    
      - This symbol is misspelled.
        Spell the symbol name correctly.
        Symbol name spelling is case-sensitive.
    
      - This symbol was added recently.
        Run `arc liberate` on the library it was added to.
    
      - This symbol is external. Use `@phutil-external-symbol`.
        Use `grep` to find usage examples of this directive.
    
    *** ALTHOUGH USUALLY EASY TO FIX, THIS IS A SERIOUS ERROR.
    *** THIS ERROR IS YOUR FAULT. YOU MUST RESOLVE IT.

              15       $this->setProxiedFuture($proxied);
              16     }
              17 
    >>>       18     qqqqqq();
              19   }
              20 
              21   public function setProxiedFuture(Future $proxied) {

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Use have/need data in ArcanistPhutilLibraryLinter.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.

(I'm going to create a new maniphest task for this so we can link to something more relevant than T1806: Move Aphront to libphutil)

Is it possible to add unit tests for this linter? *unsure*

I don't see a practical way to do it. I think this break was kind of a fluke.

This revision is now accepted and ready to land.Jul 17 2014, 9:19 PM
epriestley updated this revision to Diff 23914.

Closed by commit rARCc6e6227ef9bf (authored by @epriestley).