On 2012-09-11, at 16:41, Stéphane Ducasse <[email protected]> wrote:

> camillo
> 
> what is "added hierarchical RPackage lookup" ?
> Why do we need that beast?

Because right now RPackageOrganizer does a wrong lookup for classes
see RPackageOrganizer>>#systemClassRenamedActionFrom:
which does a 
        #packageNamed: class category
which fails if the class is not in a toplevel package.

Let's say I refactor a class TestFoo in the category 'Tests-Foo-Bar'
the message send to #packageNamed: will try to find a package named
'Tests-Foo-Bar' but the actual existing package is only 'Tests' and
the rest is the tag part.

=> you have to find the RPackage with the most common name
lookup? 'Tests-Foo-Bar' => fail!
lookup? 'Tests-Foo'     => fail!
lookup? 'Tests'         => success!
        'Tests' => lookup tag? 'Foo-Bar' => success!

Reply via email to