[#1919] SVN Log parsing of Author
Summary SVN Log parsing of Author
Queue Horde Framework Packages
Type Bug
State Resolved
Priority 1. Low
Owners
Requester shimmanning@gmail.com
Created 2005-05-05 (5785 days ago)
Due
Updated 2005-05-06 (5784 days ago)
Assigned
Resolved 2005-05-06 (5784 days ago)
Milestone
Patch No

Comments
shimmanning@gmail.com 2005-05-05 20:21:39
trying to browse some SVN repositories with CHORA will only display 
the directories and no the folders.  I have tracked this down to the

lib/Horde/VC/svn.php file.



while trying to parse the svn change log for a file, there seems to be 
a slight error in the regular expression around line 730

  if (preg_match('/^r([0-9]*) \| ([^ ]*) \| (.*) \(.*\) \| ([0-9]*) 
lines?$/', $line, $matches)) {



If the author name contains a space this will not work.

changing the expressio to read:



  if (preg_match('/^r([0-9]*) \| (.*) \| (.*) \(.*\) \| ([0-9]*) 
lines?$/', $line, $matches))



will fix the problem.  (changed [^ ]* to .* )



This issue occurs if the repository does not have an Authors name (ie 
'no author') also.



Strange how this has gone so long unchecked..

Jan Schneider <jan@horde.org> 2005-05-06 09:23:39
Fixed, thanks.