On 5/22/13, Anze Staric <[email protected]> wrote:
> A number of errors is caused by the following change
>
> r1467339 | jdreimann | 2013-04-12 18:33:52 +0200 (Fri, 12 Apr 2013) | 3
> lines
>
> Updated styling of comment history, towards #471.
> Changed styling of attachment download icon to use background-image
> instead of an img inserted on the page.
> Removed outdated screenshot.png because it doesn't appear to be used.
>
> Index: trac/trac/wiki/formatter.py
> ===================================================================
> --- trac/trac/wiki/formatter.py (revision 1467338)
> +++ trac/trac/wiki/formatter.py (revision 1467339)
> @@ -707,7 +707,7 @@
>          local_url = self.env.project_url or \
>                      (self.req or self.env).abs_href.base
>          if not url.startswith(local_url):
> -            return tag.a(tag.span(u'\u200b', class_="icon"), text,
> +            return tag.a(text, tag.span(u'\u200b', class_="icon"),
>
> If I revert it, only 4 tests fail on python 2.6.
> IMHO, the only way to fix tests failing because of this change is to
> change the expected result in tests.
>
>

we can

  1. keep the <span /> where it used to be
  2. rewrite all those test cases ... and similar instancess in
      plugins (if any) ... and ...

-- 
Regards,

Olemis.

Reply via email to