[Framework-Team] Re: PLIP #234 Review Revisions

Raphael Ritz raphael.ritz at incf.org
Thu Feb 12 14:10:40 UTC 2009


Martin Aspeli wrote:
> [..]
> FWIW, I think much of Calvin's work could've gone into a 3.2.x release 
> as bug fixes. If he doesn't break tests, and writes a few tests for 
> truly new code, then I think that's probably sufficient in most places.

Personally, I agree with the sentiment that this is mostly
a bug fix in nature.
While I still plan to take a look on Saturday I won't vote
it down for what's being discussed most recently here.

Raphael


>
> Having done something similar in the past (but not read Calvin's diff 
> in detail), I suspect most of his changes were simply to stop people 
> making use of portal_url() when they should've used 
> get_navigation_root(). There may be cases when we could add a "defect" 
> type test to show that the navigation root didn't work before, but now 
> does, but let's not create too much work for what is, in many cases, 
> more about analysing a problem and applying a few surgical fixes, than 
> writing a ton of new code.
>
> Martin
>





More information about the Framework-Team mailing list