[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] bug(let) in mod_dav_svn: Cannot navigate from repository top to repository list when SVNListParentPath is on.

From: Daniel Shahaf <d.s_at_daniel.shahaf.co.il>
Date: Fri, 4 Apr 2008 13:02:52 +0300

Filed as issue #3159. Jan, thanks for your patience getting the stylistic
nits out.

http://subversion.tigris.org/issues/show_bug.cgi?id=3159

Daniel

Jan Nijtmans wrote on Fri, 28 Mar 2008 at 11:19 +0100:
> (trying again)
> Reproduction recipe.
>
> - Go with your browser (IE or any other) to the top of any svn repository
> where SVNListParentPath is set to on, e.g.:
> <http://svn.webwideconsulting.com/plugins/friendship_plugin/>
> - Note that the super-directory (/plugins/) is browsable as well, but
> you cannot navigate to it from here.
> - I would expect that the first entry of the list is a hyperlinked "..",
> which would navigate to its super-directory.
>
> I checked this with the Collabnet win32 build of SVN 1.5.0-beta1,
> and the buglet is still there.
>
> Rationale: in repos.c, the decision whether to include the link or not
> is based on
> two things:
> - If repos_path is of length 1 (so it must be "/"), then skip this step
> - If the collection is of type DAV_SVN_RESTYPE_PARENTPATH_COLLECTION
> (the highest browsable directory), then skip this step too.
>
> This can be fixed by ignoring the first condition when
> SVNListParentPath is on.
> Unfortunately I don't have an environment set up to build mod_dav_svn to test
> this, but I'm pretty confident that this fix will work (for what's worth).
>
> Can I file an issue for this? See patch below.
>
> Regards,
> Jan Nijtmans
>
> Here is the patch against svn-1.5.0-beta1 (I re-ordered it a little because
> dav_svn__get_list_parentpath_flag() is a more expensive test, so it should
> be done last):
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-04 12:05:18 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.