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

Summary	Allow more intuitive view of repository revisions for non-VCS-aware users
Queue	Chora
Queue Version	2.0.1
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester	jon (at) spriggs (dot) org (dot) uk
Created	05/12/2008 (417 days ago)
Due
Updated	07/05/2008 (363 days ago)
Assigned
Resolved	07/05/2008 (363 days ago)
Attachments	chora.diff
Milestone
Patch	No

07/05/2008	Chuck Hagenbuch	Comment #10 Assigned to Chuck Hagenbuch State ⇒ Resolved	Reply to this comment
Committed, thanks!

07/05/2008	CVS Commit	Comment #9	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/chora/config/conf.xml?r1=1.13&r2=1.14&ty=u http://cvs.horde.org/diff.php/chora/docs/CHANGES?r1=1.126&r2=1.127&ty=u http://cvs.horde.org/diff.php/chora/templates/directory/file.inc?r1=1.22&r2=1.23&ty=u

07/04/2008	jon (at) spriggs (dot) org (dot) uk	New Attachment: chora.diff

07/03/2008	Jan Schneider	Comment #8	Reply to this comment
No patch has been uploaded.

07/03/2008	jon (at) spriggs (dot) org (dot) uk	Comment #7	Reply to this comment
As requested (and you learn something new every day!)

07/02/2008	Chuck Hagenbuch	Comment #6	Reply to this comment
Can you attach the patch as a unified diff (upload it as an attachment to the ticket instead of pasting it)?

05/23/2008	jon (at) spriggs (dot) org (dot) uk	Comment #5	Reply to this comment
Do you have other changes/ideas that affect the app because of this outlook? Not as yet. Take-up has been slow and local integration problems (due to network segregation and configuration issues with autoversioning in subversion) has lessened interest. I hope to be in a position to ramp this back up in a few weeks once a few key people are available.

05/23/2008	Chuck Hagenbuch	Comment #4	Reply to this comment
Interesting, I don't know of anyone using Chora as a document store. Do you have other changes/ideas that affect the app because of this outlook?

05/13/2008	jon (at) spriggs (dot) org (dot) uk	Comment #3	Reply to this comment
This change is predominantly for use as a documentation store, where the document is the most important aspect of the system, and it's only the document store administrators who need to know about the version number, the last person who committed it and the log. The file browser would be acceptable, if we didn't need to occasionally see the history.

05/13/2008	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
You already get the latest version if you click on the version number, and I'm not sure of how much use Chora is for people who don't know about version control systems? If you want a file browser, use Gollem.

05/12/2008	jon (at) spriggs (dot) org (dot) uk	Comment #1 Patch ⇒ Milestone ⇒ Summary ⇒ Allow more intuitive view of repository revisions for non-VCS-aware users Type ⇒ Enhancement Queue ⇒ Chora Priority ⇒ 1. Low State ⇒ New	Reply to this comment
Index: chora/config/conf.xml =================================================================== RCS file: /repository/chora/config/conf.xml,v retrieving revision 1.13 diff -r1.13 conf.xml 95a96,104 [Show Quoted Text - 9 lines][Hide Quoted Text] <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.