Your message dated Sat, 03 Apr 2010 12:54:57 +0200
with message-id <[email protected]>
and subject line 392588 fixed by upstream
has caused the Debian Bug report #392588,
regarding moin: does not shorten userlinks, as done for other links
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
392588: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=392588
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: moin
Severity: minor
Tags: patch
Hello,
Moinmoin does not shorten userlinks as it does for the current link, and
the links in the trailbar. This breaks the display on most themes, so it
is recommended to shorten them.
Attached is a fix that makes userlinks to be shortened, too.
P.S.: If there is another bug that fixes this, I am sorry for the
duplication, but I have wrote this BR while offline.
-- System Information:
Debian Release: testing/unstable
APT prefers unstable
APT policy: (900, 'unstable'), (100, 'experimental')
Architecture: powerpc (ppc)
Shell: /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18-1-powerpc
Locale: LANG=ro_RO.UTF-8, LC_CTYPE=ro_RO.UTF-8 (charmap=UTF-8)
diff -rN -u old-moinmoin/moin-1.5.3/MoinMoin/theme/__init__.py
new-moinmoin/moin-1.5.3/MoinMoin/theme/__init__.py
--- old-moinmoin/moin-1.5.3/MoinMoin/theme/__init__.py 2006-10-12
02:29:35.000000000 +0300
+++ new-moinmoin/moin-1.5.3/MoinMoin/theme/__init__.py 2006-10-12
02:29:49.000000000 +0300
@@ -369,6 +369,8 @@
for text in userlinks:
# Split text without localization, user knows what he wants
pagename, link = self.splitNavilink(text, localize=0)
+ title = self.shortenPagename(pagename)
+ link = Page(request, pagename).link_to(request, title)
if not pagename in found:
if pagename == current:
cls = 'userlink current'
--- End Message ---
--- Begin Message ---
fixed 392588 1.7.1-2
thanks
I review this BR, so you wanted to shorten the userlinks, not the user
name... this was a sensible request. I shouldn't have tagged it
wont-fix.
Upstream fixed in meanwhile! (somewhere before 1.7).
I am therefore closing this bug.
Franklin
--- End Message ---
