You are here: Foswiki>Tasks Web>Item13995 (29 May 2016, GeorgeClark)Edit Attach

Item13995: Search $changes renders a simple summary.

Priority: Urgent
Current State: Closed
Released In: 2.1.1
Target Release: patch
Applies To: Engine
Component: SEARCH
Branches: master Release02x01 Item13897 Item14033
Reported By: GeorgeClark
Waiting For:
Last Change By: GeorgeClark
The bug is down in
-            $out =~ s/\$changes(?:\(([^\)]*)\))?/
-              $item->summariseChanges(Foswiki::Store::cleanUpRevID($1), $revNum)/ges;

If $changes is used, without a requested revision, the $1 is undefined. cleanUpRevID(undef) returns zero, and summariseChanges becomes a simple summary.

The fix is relatively easy, but this is going to break any search using $changes inside a table. The $summary renders without newlines, where as $changes generates a diff, and any newlines break the table.

There are multiple tasks dealing with the rather ineffective $changes option.
  • No way to get a simple text summary, equivalent to $summary, except for the differences between the revisions See Item1390
  • No way to ask for just the prior rev. See Item9464 $changes(n) asks for differences since rev n, which is hard to predict in a search.

-- GeorgeClark - 27 Feb 2016


ItemTemplate edit

Summary Search $changes renders a simple summary.
ReportedBy GeorgeClark
Codebase 2.1.0, 2.1.0 beta1, 2.0.3, 2.0.2, 2.0.1, 2.0.0, 1.1.10, 1.1.9, trunk
SVN Range
AppliesTo Engine
Component SEARCH
Priority Urgent
CurrentState Closed
Checkins distro:8c46a18bb4e2 distro:4379bceb6ef2
TargetRelease patch
ReleasedIn 2.1.1
CheckinsOnBranches master Release02x01 Item13897 Item14033
masterCheckins distro:8c46a18bb4e2 distro:4379bceb6ef2
ItemBranchCheckins distro:8c46a18bb4e2 distro:4379bceb6ef2
Release02x01Checkins distro:4379bceb6ef2
Topic revision: r7 - 29 May 2016, GeorgeClark - This page was cached on 23 Nov 2017 - 00:46.

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