[Tickets #6690] Allow more intuitive view of repository revisions for non-VCS-aware users

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6690
-----------------------------------------------------------------------
 Ticket             | 6690
 Created By         | jon at spriggs.org.uk
 Summary            | Allow more intuitive view of repository revisions for
non-VCS-aware
                    | users
 Queue              | Chora
 Version            | 2.0.1
 Type               | Enhancement
 State              | New
 Priority           | 1. Low
 Milestone          |
 Patch              |
 Owners             |
-----------------------------------------------------------------------


jon at spriggs.org.uk (2008-05-12 05:27) wrote:

Index: chora/config/conf.xml
===================================================================
RCS file: /repository/chora/config/conf.xml,v
retrieving revision 1.13
diff -r1.13 conf.xml
95a96,104
>
>
>   <configboolean name="show_revision_on_filename_click"
> desc="The default action when clicking on a file
> in the browse view is to show the revision history of that file,
> however, for those who are not used to version control systems, this is
> not intuitive. Unticking this box makes the action of clicking on a
file
> to display the contents of the file, and clicking on the revision
number
> shows the revision history.">true</configboolean>

Index: chora/templates/directory/file.inc
===================================================================
RCS file: /repository/chora/templates/directory/file.inc,v
retrieving revision 1.22
diff -r1.22 file.inc
9c9,14
<    <a href="<?php echo $url ?>"><?php echo $name ?></a>
---
> <?php if ($conf['show_revision_on_filename_click']): ?>
>    <a href="<?php echo $url ?>">
> <?php else: ?>
>    <a href="<?php echo Chora::url('co', $fileName, array('r' => $head))
?>">
> <?php endif; ?>
> <?php echo $name ?></a>
12c17,23
<    <strong><a href="<?php echo Chora::url('co', $fileName, array('r' =>
$head)) ?>"><?php echo $head ?></a></strong>
---
>    <strong>
> <?php if ($conf['show_revision_on_filename_click']): ?>
>    <a href="<?php echo Chora::url('co', $fileName, array('r' => $head))
?>">
> <?php else: ?>
>    <a href="<?php echo $url ?>">
> <?php endif; ?>
> <?php echo $head ?></a></strong>

I don't know whether this is the right patch format, but I'd be glad to do
whatever is needed to get this in, as I think this would be a useful
addition.