Developer discussion for Creative Commons technology and tools

["SourceForge.net" <noreply AT sourceforge.net>]

Bugs item #2142227, was opened at 2008-10-02 04:04
Message generated for change (Comment added) made by fourstones
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=559966&aid=2142227&group_id=80503

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: cchost
Group: cchost5_ccM3
Status: Closed
Resolution: Invalid
Priority: 5
Private: No
Submitted By: ben_weiner (ben_weiner)
Assigned to: Victor Stone (fourstones)
Summary: ID3 tags page broken due to missing class

Initial Comment:
URL: http://openfontlibrary5.localhost/admin/id3tags

Here's my fix (as I'm an end-user I've installed non-versioned source and 
then versioned the whole tree so I can learn about the codebase, so the 
revision number is mine, sorry)

Hope I have the bug correctly tagged.

Cheers,

--
Ben


--- cchost_lib/cc-filetagger.php        (revision 4)
+++ cchost_lib/cc-filetagger.php        (working copy)
@@ -223,6 +223,7 @@
     */
     function AdminTagger()
     {
+        require_once('cchost_lib/cc-page.php');
         CCPage::SetTitle("Configure ID3 Tagger");
         require_once('cchost_lib/cc-filetagger-admin.inc');
         $form = new CCAdminTaggerForm($this);


----------------------------------------------------------------------

>Comment By: Victor Stone (fourstones)
Date: 2008-10-02 06:26

Message:
thanks ben, then it's a 'bug' in HACKING. I've updated that file and
checked it in.

The distro information files are a snapshot, for the latest information
you could always check the wiki 

http://wiki.creativecommons.org/cchost

----------------------------------------------------------------------

Comment By: ben_weiner (ben_weiner)
Date: 2008-10-02 06:09

Message:
OK, sorry, I checked out a different release. I have:

URL: https://svn.sourceforge.net/svnroot/cctools/cchost/trunk

I got the link from the HACKING file in the 'distribution' version :(

Thanks for the update. I will export from that.

----------------------------------------------------------------------

Comment By: Victor Stone (fourstones)
Date: 2008-10-02 05:02

Message:
Code is fine in the Creative Commons trunk
http://code.creativecommons.org/svnroot/cchost/trunk/cchost_lib/cc-filetagger.php

----------------------------------------------------------------------

Comment By: ben_weiner (ben_weiner)
Date: 2008-10-02 04:29

Message:
Have just checked that this file is unchanged at current revision.

----------------------------------------------------------------------

Comment By: ben_weiner (ben_weiner)
Date: 2008-10-02 04:07

Message:
Hi

Attached a file version of the patch as the indenting was lost.

Ben
File Added: cc-filetagger.php.patch

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=559966&aid=2142227&group_id=80503