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

Dave Hall <[email protected]>
Newsgroups gmane.comp.web.phpgroupware.tracker
Message-ID <[email protected]>
Update of patch #5387 (project phpgroupware):

                  Status:          Ready For Test => In Progress            
             Assigned to:                     ceb => Caeies                 

    _______________________________________________________

Follow-up 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

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?5387>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.