• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

Item5139: FindElsewherePlugin no longer find acronym type topics.

Item Form Data

AppliesTo: Component: Priority: CurrentState: WaitingFor: TargetRelease ReleasedIn
Extension FindElsewherePlugin Normal Closed   n/a  

Edit Form Data

Summary:
Reported By:
Codebase:
Applies To:
Component:
Priority:
Current State:
Waiting For:
Target Release:
Released In:
 

Detail

FindElsewherePlugin no longer find acronym type topics.

It is the findTopicElsewhere() function change that does not seem to look for acronym (all uppercase) topics any longer.

If this gets fixed then I would propose that the plugin gets a setting so you can turn the acronym finding off.

I found this bug when I was about to do my normal hack of this plugin. But I assume that old users of this plugin may want the old functionality.

The reason I turn it off is that it looks awful when the same acronym is linked 50 times in a topic.

-- KennethLavrsen - 17 Dec 2007

Acronyms works alright here - could you elaborate on the testcase? (I just created ACRONYM in Main and the plugin page had it show up alright).

I appreciate the thought on multiple linking, will give a "first occurence only"-feature a shot.

-- TWiki:Main.SteffenPoulsen - 17 Dec 2007

One-link-only feature added, closing.

-- TWiki:Main.SteffenPoulsen - 18 Dec 2007

This destroys the whole idea of the Plugin and totally goofs up the way we use it.

E.g. we sign without the Main. prefix.

We write [[ALT]] to link to the acronym web in documents that are 30 pages long.

When you write a WikiWord or use a square bracket link - then you expect a link.

I am reverting this.

If you add it again then please make it configurable so you can turn that feature on. And leave it by default off.

But honestly - I doubt anyone want a feature where the effect is that WikiWords with a topic in current web are always linked. And WikiWords that are in a find-elsewhere web are only linked once. It scores up round 5 or 6 on a TWiki:Codev.NerdoMeter.

The find once makes sense with Acronyms written without any square brackets because you can easily use such an acronym 10 times in a topic but you only want it linked once.

This is why I modified the plugin so that ALT is not linked but [[ALT]] is without having to append it with the Acronym web name. In the case of simple acronyms the find once feature makes sense. But people may have searches that return acronyms they want linked every time so such a change in spec has to be configurable by plugin setting.

-- KennethLavrsen - 18 Dec 2007

By the way. On my home installation the plugin works in the old version. Also for acronyms.

So I must have goofed up in the configuration at work. Setting to No Action Required.

-- TWiki:Main.KennethLavrsen - 18 Dec 2007

I think the bug you have stumbled into might have been that acronyms with alternate linktext were no longer found elsewhere, i.e. [[ACRONYM][link text]] - should be fixed now.

I don't personally make a distinction between acronyms and wikiwords, they are equally important / linkable to me - but we could use the dev topic to find out whether there's any consensus here.

Perhaps a minispec for the one-time-only linking could be to only skip linking if

  • FINDACRONYMSONLYONCE is true
  • It is a non-forced acronym (not in square brackets)
  • It has already been linked

I think that should be safe and aligned with what you suggest - but I am open to input smile

"But people may have searches that return acronyms they want linked every time so such a change in spec has to be configurable by plugin setting." - we are not talking about Item4748 here, are we? *blink*.

-- SteffenPoulsen - 18 Dec 2007

ACRONYM option implemented according to minispec.

-- TWiki:Main.SteffenPoulsen - 19 Dec 2007

Edit | Attach | Watch | Print version | History: r19 < r18 < r17 < r16 < r15 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r19 - 2007-12-19 - SteffenPoulsen
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback