• Resolved anemoone

    (@anemoone)


    Hello.

    I’ve installed the theme. I’ve noticed two problems.

    Comments lack commentators’ names nad dates.

    The other problem was in the customizable footer (via Theme Options). The links did’t link properly, so I had to unlink some words.

    http://wordpress.org/extend/themes/museum-core/

Viewing 15 replies - 1 through 15 (of 19 total)
  • Michael

    (@alchymyth)

    in functions.php of the theme, nearly half-way down:
    add echo to this line (shown after the edit):

    <?php echo sprintf(__('On %1$s at %2$s %3$s said:','museum-core'), get_comment_date(), get_comment_time(), get_comment_author_link()) ?>
    Thread Starter anemoone

    (@anemoone)

    Thank you.

    I’ve added the word. Now comments are OK.

    Theme Author Chris Reynolds

    (@jazzs3quence)

    Sorry about that guys. I didn’t have emails set up on this tag yet. As soon as I get SVN access to update the theme, I’ll take care of this. I think I found and fixed the customizable footer, but I’ll take a look at that, too. For now, the latest version is up on GitHub. I’ve added these as tickets in the issue tracker there: https://github.com/jazzsequence/AP-Museum_Core

    Theme Author Chris Reynolds

    (@jazzs3quence)

    @anemoone The issue with the footer links has been found and will be fixed in the next version. For now, if you want to fix it, open footer.php and find this:

    <div class="credit">
    				<?php if ( $options['footer'] != '' ) {
    				echo $options['footer'];
    				} else {
    					echo $defaults['footer'];
    				} ?>
    			</div>

    and replace that with this

    <div class="credit">
    				<?php if ( $options['footer'] != '' ) {
    				echo stripcslashes($options['footer']);
    				} else {
    					echo $defaults['footer'];
    				} ?>
    			</div>

    That should take care of it — there’s some other stuff I’m fixing in the theme-options.php but that should resolve the root issue (assuming your links were doing what mine were, which was trying to link to http://mydomain.com/"http://mynewlink.com/")

    Thread Starter anemoone

    (@anemoone)

    @chris Reynolds

    assuming your links were doing what mine were, which was trying to link to http://mydomain.com/”http://mynewlink.com/&#8221;

    Yes, that’s the same error.

    I’ve changed that fragment of code.
    Now it’s fine. Thanks.

    Theme Author Chris Reynolds

    (@jazzs3quence)

    This will be fixed in the next version which I’ve uploaded today.

    Edit: …and it’s now live http://wordpress.org/extend/themes/museum-core/

    Thread Starter anemoone

    (@anemoone)

    When I updated to the newest version, this warning appeared on my page:

    “Warning: sprintf() [function.sprintf]: Too few arguments in /home/anemoone/public_html/wp-content/themes/museum-core/functions.php on line 335”

    So I replaced functions.php file with the the same file from the earlier version of the theme, found in Theme SVN.

    Theme Author Chris Reynolds

    (@jazzs3quence)

    Looking now.

    Theme Author Chris Reynolds

    (@jazzs3quence)

    found it.

    Line 335 in functions.php.

    Was:

    'footer' => sprintf( __( '%1$s %2$s %3$s . ', 'museum-core' ), '©', date('Y'), get_bloginfo('title') ) . sprintf( __( 'Museum Core by %1$sMuseum Themes%2$s is proudly powered by %4$sWordPress%2$s.', 'museum-core' ), '<a href="http://museumthemes.com/" target="_blank" title="Museum Themes">', '</a>', '<a href="http://wordpress.org" target="_blank">' ),

    Should be:

    'footer' => sprintf( __( '%1$s %2$s %3$s . ', 'museum-core' ), '©', date('Y'), get_bloginfo('title') ) . sprintf( __( 'Museum Core by %1$sMuseum Themes%2$s is proudly powered by %3$sWordPress%2$s.', 'museum-core' ), '<a href="http://museumthemes.com/" target="_blank" title="Museum Themes">', '</a>', '<a href="http://wordpress.org" target="_blank">' ),

    Theme Author Chris Reynolds

    (@jazzs3quence)

    fixed in 1.0.7.2

    Thread Starter anemoone

    (@anemoone)

    Installed that version. No problems.

    Theme Author Chris Reynolds

    (@jazzs3quence)

    excellent. thanks for the feedback. Helped me clean out the bugs. 🙂

    Thread Starter anemoone

    (@anemoone)

    There’s one more thing. Why pignback from my site didn’t appear below the text?

    Don’t know if it is the theme’s fault.

    http://kratery.com/2012/04/16/smolensk-impresje/

    Theme Author Chris Reynolds

    (@jazzs3quence)

    Hm. Not sure. I’ll look into it.

    Want to help out with a Polish language file for the theme?

    Thread Starter anemoone

    (@anemoone)

    Want to help out with a Polish language file for the theme?

    Of course I could help, but have little experience. I translated for myself most of the words in two themes I used before (in my editor). Didn’t prepare any files.

Viewing 15 replies - 1 through 15 (of 19 total)
  • The topic ‘[Theme: Museum Core] Comments’ is closed to new replies.