• 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.

Test case:

http://develop.twiki.org/~develop/cgi-bin/search/Sandbox?search=testing

shows summaries with linked text.

Possibly caused by removing nop tags early, then render text again?

-- PTh

no, this was an intentional enhancement.

-- SD

I call this a bug since it is a spec change that might break existing apps. Consider for example if the RSS feeds summary suddenly contain links. Better to add a new switch to $summary() if you want that functionality, such as $summary(render) (which should be treated as an enhancement).

-- PTh

RSS feeds are one of the major reasons i made this change! if you look at almost any modern blog with a modern blog reader, you get html - the feed sends escaped html, and so does twiki now)

Its a change that should be harmless, and had been desired for years - I finally bit the bullet and made summary more wikilike - if we always have to add new syntax to improve things, twiki's going to get even more crufty than it already is - we really could move quite a bit of old syntax into a legacy package...

SD

I documented this change in DakarReleaseNotes. SVN 1801

CC

Reopening this. Please compare the result of WebSearch and plain search script. Should look the same.

- PTh

Peter, you assert above that this change might break an existing TWiki application. It's an obvious and attractive enhancement, so it would be really good to know what application you have in mind that it might break? Perhaps if you could illustrate how it can break something important, then Sven might feel motivated to revert all his hard work. Or is it just that you feel search should just be a CGI API to %SEARCH? Otherwise I can't understand your reasoning.....?

CC

One feature at a time please. There is no need to revert the hard work, I created enhancement request Item1376: Add new $summary(render) to FormattedSearch. See that topic why we should not change the spec of $summary.

Actually, thinking it over, TWiki applications are unlikely to use the search script, so there is a low risk that this bug Item1307 breaks them. TWiki applications make heavy use of %SEARCH{}%, and the $summary() in %SEARCH{}% works as expected (except for the small bug Item1306, Web prefix not removed from summary).

This Item1307 is a bug since one expect that the output of search script with =search?format=something and %SEARCH{ format="something" }% (delivered by view script) would be identical.

-- PTh

search and WebSearch now seem to uptput the same thing, but truly is all fakery. they 2 of them do not seem to use the same code paths.

Worse, the 1-2 weeks work i did to make html rendering working for xml, html plain text and so in have been removed and killed over the last 3 months, so i'm giving up.

svn:8227

SD

ItemTemplate
Summary In search script, summary should not render links
ReportedBy PeterThoeny
Codebase

SVN Range Tue, 03 Jan 2006 build 8080
AppliesTo Engine
Component

Priority Normal
CurrentState Closed
WaitingFor

Checkins 8101 8227
Edit | Attach | Watch | Print version | History: r11 < r10 < r9 < r8 < r7 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r11 - 2006-01-11 - SvenDowideit
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2018 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback