Martin Waitz wrote:
> hoi :)
>
> On Thu, Aug 17, 2006 at 11:41:43AM +0200, Jakub Narebski wrote:
>> > Project list and OPML generation are now hooked into the list as
>> > "/summary" and "/opml".
>>
>> It would be better to use "/list" or "/projects", although empty or
>> undef or "/" action would work as well.
>>
>> Having both "/summary" (for projects list) and "summary" (for project
>> summary) is bad, bad idea.
>
> well, it was a little trick so that only one default action "summary"
> is needed. I have no strong feelings about it so any other name is
> good for me, too.
if (defined $project) { # with the patch that undef $project when empty
$action ||= "summary";
} else {
$action ||= "/projects_list"; # or simply "projects_list", or "/"
}
--
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html