#151 ID3 tags page broken due to missing class

[ben_weiner]

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);

[ben_weiner]

Hi

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

Ben
File Added: cc-filetagger.php.patch

[ben_weiner]

patch text snippet

[ben_weiner]

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

[Victor Stone]

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

[ben_weiner]

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.

[Victor Stone]

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