According to the SmiliesPlugin documentation:
   * The name of the topic containig _[sic]_ your Smilies. Default: TWiki.SmiliesPlugin 
   * #Set TOPIC = SmiliesPlugin 

The benefit of using a separate topic is to retain changes after applying a patch or upgrade. Unfortunately that feature doesn't work: if I set SMILIESPLUGIN_TOPIC to my own topic, the short cut and alt text from the new topic are recognized. But the image is "broken" unless it is attached to the original plugin topic.

Here's the fix (I don't have access to check it in, or time to learn how to run the test cases). The topic name is picked up from the preferences (if used) and parsed into $web and $topic. But then those variables aren't used to create $smiliesPubUrl:
    $topic =
      TWiki::Func::getPreferencesValue( 'SMILIESPLUGIN_TOPIC' ) 
          || "$installWeb.SmiliesPlugin";
    $web = $installWeb;
    if( $topic =~ /(.+)\.(.+)/ ) {
        $web = $1;
        $topic = $2;
[... elided ...]

    #$smiliesPubUrl =
    #  TWiki::Func::getUrlHost() . TWiki::Func::getPubUrlPath() .
    #      "/$installWeb/SmiliesPlugin";

    $smiliesPubUrl = TWiki::Func::getUrlHost() . TWiki::Func::getPubUrlPath() . "/$web/$topic"; 

P.S. I think I'm using the current version of the plugin. On the plugin topic itself, it says "Plugin Version 21 Nov 2007".
But on InstalledPlugins, it says (Dakar, $Rev: 16049 (11 Aug 2008) $)

-- SeanMorgan - 12 Nov 2008

ItemTemplate edit

Summary SmiliesPlugin TOPIC variable not used for images
ReportedBy TWiki:Main.SeanMorgan
Codebase trunk
SVN Range TWiki-4.2.3, Wed, 06 Aug 2008, build 17396
AppliesTo Extension
Component SmiliesPlugin
Priority Normal
CurrentState Closed
Checkins distro:27737ca54936 distro:52dde63baf72 distro:fd95ce1ec2da
TargetRelease patch
ReleasedIn 1.0.0
Topic revision: r9 - 08 Jan 2009, KwangErnLiew - This page was cached on 13 Apr 2017 - 12:10.

The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License