WordPress.org

Ready to get started?Download WordPress

Forums

[Plugin: Subscribe to Comments] Bug on line 826 (11 posts)

  1. mranderson
    Member
    Posted 4 years ago #

    For anyone who wants to localize this excelent plugin, there's a bug on line 826 that makes impossible localization/internationalization

    Line 826 should be like this

    load_plugin_textdomain('subscribe-to-comments','/wp-content/plugins/subscribe-to-comments/');

    POT file is in extra folder, but localized PO files must be on subscribe-to-comments folder for this to work

    I hope it will help to someone, or at least the author upgrades the plugin. Greetings.

    http://wordpress.org/extend/plugins/subscribe-to-comments/

  2. philippetrov
    Member
    Posted 4 years ago #

    and another unreleated bug - on line 1084

    <p><a href=“<?php echo add_query_arg(’showccfield’, ‘1′); ?>“>

    must be replaced with:

    <p><a href="<?php echo attribute_escape(add_query_arg('showccfield', '1')); ?>">

    This fixes some XHTML validation issues

  3. Alexander Gieg
    Member
    Posted 4 years ago #

    @philippetrov: thanks for the small correction. Someone really should take the unnoficial 2.1.3 (which also solves mranderson's corrected line), add this, then upload the result as a new plugin...

  4. bramirez
    Member
    Posted 4 years ago #

    Can someone explain me exactly what I need to do to localize the plugin? Do I only need to translate the content of subscribe-to-comments.pot where it says msgid???

  5. Alexander Gieg
    Member
    Posted 4 years ago #

    The easiest way to make translations is by using the Codestyling Localization plugin. It allows you to translate directly in WordPress, without the need to download and upload files to test things.

    Once you're done, if you wish to contribute your translation to the author you can simply download the resulting .mo and .po files and send them to the plugin author for him to include it in the next version.

    See the plugin's official site for detailed instructions. Although they describe a quite old version of the plugin, they're still useful.

  6. Alexander Gieg
    Member
    Posted 4 years ago #

    PS.: It has one extra advantage: with it you can delete translations you don't need, thus wasting less disk space. Although a small thing, this is a nice touch for those whose blogs are hosted on ISPs that charge for storage or provide small disk quotas.

  7. stuart001
    Member
    Posted 4 years ago #

    Uhhhhh . . . bugs on line 826 and line 1084? POT file? My download for WP 2.9.2 doesn't have a POT file, and when I open subscribe-to-comments-now.php in Dreamweaver there are a total of 84 lines. I can't get this plugin to work on WP 2.9.2. Any suggestions? Thanks. -Stuart

  8. Rhand
    Member
    Posted 3 years ago #

    @ mranderson. Made the changes on line 826, but my subscribe-to-comments-nl.mo in /httpdocs/wp-content/plugins/subscribe-to-comments is still not loaded.

  9. Rhand
    Member
    Posted 3 years ago #

    All good now. Had to add some translations into the configuration area in the WP Dashboard.

    PS Have not tested plugin after hack as of yet

  10. codestyling
    Member
    Posted 3 years ago #

    The Autor uses this to load the textdomain (translation file):

    load_plugin_textdomain('subscribe-to-comments');

    This leads to the result, that the file will be expected at the main folder of the plugin. If you placed it there instead inside the "extras" subfolder, it would have been loaded.
    But my plugin "Codestyling Localization" will generate translations at this subfolder anyway because of existance of *.pot file there and assumes load from this point. But this won't happen as long as the load call won't be changed into:

    load_plugin_textdomain('subscribe-to-comments', false, 'subscribe-to-comments/extras');

    to avoid deprecated warnings since WP2.7 and allow online translations.

  11. camu
    Member
    Posted 3 years ago #

    If you're interested in alternatives, I've rewritten the code from scratch and I'm currently working in adding new features:

    wordpress.org/extend/plugins/subscribe-to-comments-reloaded

    Check it out!
    Camu

Topic Closed

This topic has been closed to new replies.

About this Topic