patchphpGroupWare - Patches: patch #5387, patch for adding support of svn in...


You are not allowed to post comments on this tracker with your current authentification level.

patch #5387: patch for adding support of svn in the api and more

Submitted by:  Caeies <Caeies>
Submitted on:  Wed Sep 13 10:08:40 2006  
Category: filemanagerPriority: 5 - Normal
Item Group: SVN trunk (dev code)Status: In Progress
Privacy: PublicAssigned to: Caeies <Caeies>
Open/Closed: Open

Tue Oct 24 14:07:35 2006, comment #4:

Sorry I was a little too eager to close this one

Dave Hall <skwashd>
Project Administrator
Tue Oct 24 13:59:55 2006, comment #3:

This has been applied, closing

Dave Hall <skwashd>
Project Administrator
Tue Sep 19 11:50:12 2006, comment #2:

Hi Dave,

Does the filemanage/templates/base/css/base.css automatically included or should we take special action ?

ok for phpdoc into the new SVN classes

For the split, That's not third party code, then it follow the other equivalent class (http_dav_client IIRC) which use the same structure. I don't see any pertinent reasons for moving it out of the class, since it's purly internal ...



Caeies <Caeies>
Project AdministratorIn charge of this item.
Sun Sep 17 01:32:09 2006, comment #1:

This looks good.

A few suggestions

Move the diff CSS to filemanage/templates/base/css/base.css and just use classes, there is no need to allow it to only be applied to td elements

Fix up the phpdocs for new SVN classes

Split the svn parser off into a new class. If it is 3rd party code put it into phpgwapi/inc/svnparser and just include it in the top of the svn_dav class

Feel free to commit it

Dave Hall <skwashd>
Project Administrator
Wed Sep 13 10:08:40 2006, original submission:

Hi all,

The patch here allow people to use svn as a repository, when it's used by apache, and autoversionning system. We add the possibility to see diff, to navigate through revision, look at logs ... You will need the svn command on the server if you want to be able to see diffs.

Then, I add to this, a lot's of E_ALL fixes, some nasty bug with group managing, and remove some dangerous code for DAV/SVN users.

Hope this help



Caeies <Caeies>
Project AdministratorIn charge of this item.


Attached Files
file #10751:  patch_for_svn.tgz added by Caeies (18kB - application/x-gtar - patch for filemanager ...)


Depends on the following items: None found

Items that depend on this one: None found


CC list is empty


Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.


Please enter the title of George Orwell's famous dystopian book (it's a date):



Follow 9 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Tue Oct 24 14:07:35 2006skwashdStatusDone=>In Progress
Tue Oct 24 13:59:55 2006skwashdStatusIn Progress=>Done
Sun Sep 17 01:32:09 2006skwashdStatusReady For Test=>In Progress
  Assigned toceb=>Caeies
Wed Sep 13 10:15:05 2006CaeiesCarbon-Copy-=>Added -unavailable-
Wed Sep 13 10:08:40 2006CaeiesAttached File-=>Added patch_for_svn.tgz, #10751
  Carbon-Copy-=>Added skwashd

Back to the top

Powered by Savane 3.1-cleanup1