Error in the theme upgrade guide Error in the theme upgrade guide
 

News:

CPG Release 1.6.26
Correct PHP8.2 issues with user and language managers.
Additional fixes for PHP 8.2
Correct PHP8 error with SMF 2.0 bridge.
Correct IPTC supplimental category parsing.
Download and info HERE

Main Menu

Error in the theme upgrade guide

Started by Bacchus, January 13, 2006, 09:24:39 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Bacchus

On the theme upgrade guide, we need to correcte the number 9 to include the need for the HOME link.

http://coppermine-gallery.net/demo/cpg14x/docs/theme/edit_theme.html

We should read something similar to this instead:

// HTML template for main menu
$template_sys_menu = <<<EOT
                <span class="topmenu">
<!-- BEGIN home -->
                      <a href="{HOME_TGT}" title="{HOME_TITLE}">{HOME_LNK}</a><br /><br />
<!-- END home -->

<!-- BEGIN my_gallery -->
                        <a href="{MY_GAL_TGT}" title="{MY_GAL_TITLE}">{MY_GAL_LNK}</a><br /><br />
<!-- END my_gallery -->

<!-- BEGIN allow_memberlist -->
                        <a href="{MEMBERLIST_TGT}" title="{MEMBERLIST_TITLE}">{MEMBERLIST_LNK}</a><br /><br />
<!-- END allow_memberlist -->

<!-- BEGIN my_profile -->
                        <a href="{MY_PROF_TGT}">{MY_PROF_LNK}</a><br /><br />
<!-- END my_profile -->


etc...

That's why I did not include them in my previous upgraded themes, I just did not know about it.

Thanks!

Tranz

Thank you for pointing out the omission. However, you might want to find out the meaning of "bug" and not use it when it does not apply.

Bacchus

Well, it's a matter of language  ;D

But I should have said error/mistake instead of bug.

Thanks.

Joachim Müller

donnovan, could you look into this please?

Joachim Müller


Paver

Since the purported example from "Hardwired" in edit_theme.html doesn't match the actual "Hardwired" theme code, I thought it was fairly straight-forward to make this fix.  I made the two match now, as suggested by Bacchus, and I committed this fix to stable & devel.