Ticket #10592 (closed defect: fixed)

Opened 7 months ago

Last modified 7 months ago

Error when generating the documentation of the 'smtp_always_cc' option

Reported by: sdegrande Assigned to: Blackhex
Priority: normal Component: DiscussionPlugin
Severity: normal Keywords:
Cc: rjollos Trac Release: 1.0

Description (Last modified by rjollos)

With r12143 on Trac1.0, on the wiki/TracIni page, there is an error report instead of the documentation string of the 'smtp_always_cc' option:

Project-Id-Version: Trac 0.13 Report-Msgid-Bugs-To: ​trac-
dev@googlegroups.com POT-Creation-Date: 2011-02-23 13:05+0900 PO-
Revision-Date: 2011-02-23 22:27+0900 Last-Translator: FULL NAME
<EMAIL@ADDRESS> Language-Team: en_US <​LL@li.org> Plural-Forms:
nplurals=2; plural=(n != 1) MIME-Version: 1.0 Content-Type: text/plain;
charset=utf-8 Content-Transfer-Encoding: 8bit Generated-By: Babel 0.9.6

The proposed attached patch fixes it.

Thanks for your great work !

Attachments

doc_error.patch (485 bytes) - added by sdegrande on 11/02/12 12:22:12.
Before.png (38.3 kB) - added by rjollos on 11/03/12 20:51:17.
After.png (40.4 kB) - added by rjollos on 11/03/12 20:51:41.
doc_error.png (58.2 kB) - added by sdegrande on 11/05/12 11:39:55.

Change History

11/02/12 12:22:12 changed by sdegrande

  • attachment doc_error.patch added.

11/02/12 21:09:58 changed by rjollos

  • description changed.

11/03/12 20:48:51 changed by rjollos

  • status changed from new to closed.
  • resolution set to fixed.

(In [12292]) Fixes #10592: Implemented missing TracIni option documentation for smtp_always_cc. Thanks to sdegrande for the patch.

11/03/12 20:51:17 changed by rjollos

  • attachment Before.png added.

11/03/12 20:51:41 changed by rjollos

  • attachment After.png added.

(follow-up: ↓ 4 ) 11/03/12 20:53:05 changed by rjollos

Before/After screen captures of discussion documentation section:

(in reply to: ↑ 3 ; follow-up: ↓ 5 ) 11/04/12 09:33:09 changed by sdegrande

Replying to rjollos:

Before/After screen captures of discussion documentation section:

How comes that you have a blank documentation (in your "before" screenshot) while I have an error msg ?

(in reply to: ↑ 4 ; follow-up: ↓ 6 ) 11/04/12 20:32:05 changed by rjollos

Replying to sdegrande:

How comes that you have a blank documentation (in your "before" screenshot) while I have an error msg ?

I'm not sure. I was testing with the latest version of the Trac trunk, running tracd. Do you have a screen capture of the error?

(in reply to: ↑ 5 ; follow-up: ↓ 8 ) 11/05/12 11:39:42 changed by sdegrande

Replying to rjollos:

I'm not sure. I was testing with the latest version of the Trac trunk, running tracd. Do you have a screen capture of the error?

Here it is :

System informations:

Trac1.0
Babel0.9.6
Docutils0.8.1
Genshi0.6 (without speedups)
mod_wsgi3.3 (WSGIProcessGroup sites WSGIApplicationGroup %{GLOBAL})
Pygments1.5
pysqlite2.4.1
Python2.6.6 (r266:84292, Dec 26 2010, 22:48:11) [GCC 4.4.5]
pytz2012b
setuptools0.6
SQLite3.7.3
jQuery1.7.2

11/05/12 11:39:55 changed by sdegrande

  • attachment doc_error.png added.

11/05/12 12:00:05 changed by rjollos

I suspect the difference is that you are using Babel and a non-English language.

(in reply to: ↑ 6 ; follow-up: ↓ 9 ) 11/05/12 12:19:32 changed by jun66j5

Replying to sdegrande:

Here it is : (image)

This issue is TracIni macro issue in Trac core with Babel. gettext and dgettext functions, which are translation functions in Babel, translate an empty string to the *.po header.

IniAdminPlugin has the same issue and it was fixed in r12306. And, I'll fix in Trac 1.0.1. Thanks.

(in reply to: ↑ 8 ) 11/05/12 14:25:44 changed by sdegrande

Replying to jun66j5:

This issue is TracIni macro issue in Trac core with Babel. gettext and dgettext functions, which are translation functions in Babel, translate an empty string to the *.po header. IniAdminPlugin has the same issue and it was fixed in r12306. And, I'll fix in Trac 1.0.1. Thanks.

Oh, my bad...

I should have trace the code more deeply, instead of thinking it was a kind of an error msg.

Sorry.

11/05/12 16:12:45 changed by rjollos

Jun, thank you for sorting out the issue for us.

11/06/12 19:50:36 changed by rjollos

Link to defect ticket that Jun opened: t:#10940.


Add/Change #10592 (Error when generating the documentation of the 'smtp_always_cc' option)




Change Properties
Action