[WebSVN] Little bug for path with log.php

[Yokav - Mailing]

Hello,

http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&isdir=1&peg=843

On this page we show all the logs of blame.php but into the path,
blame.php is marked as rev 843.

For doing this, display the log of a file at previous rev than the last
one then click on "Go to the most recent rev" and you will see the small
bug.

------------------------------------------------------
http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2410657

To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].

[Quinn Taylor]

Well, yes and no. In the upper right, you see rev 910 — the 843 in the  
upper left is the peg revision, which you can see specified in the  
URL. (The peg revision is passed in case the path changes between  
revisions.) Also, I'm not sure why the "isdir" parameter is set to  
true. This URL should display exactly what you expect:

http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php

I traced the steps you mention, and I got the peg rev parameter, but  
not isdir. It might be worth checking how you got to that URL and  
seeing if we can streamline it at all, or correctly the mistaken link  
that marked it as a directory.

  - Quinn

On Oct 23, 2009, at 7:19 AM, Yokav wrote:

> Hello,
>
> http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&isdir=1&peg=843
>
> On this page we show all the logs of blame.php but into the path,
> blame.php is marked as rev 843.
>
> For doing this, display the log of a file at previous rev than the  
> last
> one then click on "Go to the most recent rev" and you will see the  
> small
> bug.

------------------------------------------------------
http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2410689

To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].

[Yokav - Mailing]

Why don't remove the peg from the url of the link "Go to the most rev"?

For isdir, open this page
http://trunk.websvn.info/revision.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&rev=843&peg=843
Then click on "View log".

Le 23/10/09 18:10, Quinn Taylor a écrit :

> Well, yes and no. In the upper right, you see rev 910 — the 843 in the
> upper left is the peg revision, which you can see specified in the
> URL. (The peg revision is passed in case the path changes between
> revisions.) Also, I'm not sure why the "isdir" parameter is set to
> true. This URL should display exactly what you expect:
>
> http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php
>
> I traced the steps you mention, and I got the peg rev parameter, but
> not isdir. It might be worth checking how you got to that URL and
> seeing if we can streamline it at all, or correctly the mistaken link
> that marked it as a directory.
>
>    - Quinn
>
> On Oct 23, 2009, at 7:19 AM, Yokav wrote:
>
>> Hello,
>>
>> http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&isdir=1&peg=843
>>
>> On this page we show all the logs of blame.php but into the path,
>> blame.php is marked as rev 843.
>>
>> For doing this, display the log of a file at previous rev than the
>> last
>> one then click on "Go to the most recent rev" and you will see the
>> small
>> bug.
>
> ------------------------------------------------------
> http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2410689
>
> To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].

------------------------------------------------------
http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2411020

To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].

[Quinn Taylor]

The reason is that if the path changed in a more recent revision, the path won't be found at the head revision. Peg revisions ("/path/to/resource@PEG") are different from operative revisions ("/path/to/resource -r REV) and work pretty much independently. A smarter SVN module could perhaps figure out that the path hasn't changed and discard the peg revision from the URL, but we don't yet have that luxury.  :-)

 - Quinn

On Oct 24, 2009, at 4:31 AM, Yokav wrote:

Why don't remove the peg from the url of the link "Go to the most rev"?

For isdir, open this page
http://trunk.websvn.info/revision.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&rev=843&peg=843
Then click on "View log".

Le 23/10/09 18:10, Quinn Taylor a écrit :

Well, yes and no. In the upper right, you see rev 910 — the 843 in the

upper left is the peg revision, which you can see specified in the

URL. (The peg revision is passed in case the path changes between

revisions.) Also, I'm not sure why the "isdir" parameter is set to

true. This URL should display exactly what you expect:

http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php

I traced the steps you mention, and I got the peg rev parameter, but

not isdir. It might be worth checking how you got to that URL and

seeing if we can streamline it at all, or correctly the mistaken link

that marked it as a directory.

  - Quinn

On Oct 23, 2009, at 7:19 AM, Yokav wrote:

Hello,

http://trunk.websvn.info/log.php?repname=WebSVN&path=%2Ftrunk%2Fblame.php&isdir=1&peg=843

On this page we show all the logs of blame.php but into the path,

blame.php is marked as rev 843.

For doing this, display the log of a file at previous rev than the

last

one then click on "Go to the most recent rev" and you will see the

small

bug.

------------------------------------------------------

http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2410689

To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].

------------------------------------------------------
http://websvn.tigris.org/ds/viewMessage.do?dsForumId=1547&dsMessageId=2411020

To unsubscribe from this discussion, e-mail: [dev-unsubscribe@...].