1

Topic: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

I've discovered few bug in mailing list pages:

1) When i list internal moderators i can't see the checkbox near the address so i can't' remove it.

3) I can't add internal members from Mail lIst profile page but if i add them by user profile it works (and i see the checkbox near the name)

3) when i delete a mail list when I see the user profile of an ex-member I see something like "Warning::Non Existent account" (see attach)

Those problems still exist in 1.5.0

Post's attachments

1.png
1.png 53.15 kb, file has never been downloaded. 

You don't have the permssions to download the attachments of this post.

2 (edited by robynhub 2011-06-21 18:34:33)

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

Second screenshot.

Post's attachments

3.png
3.png 39.56 kb, file has never been downloaded. 

You don't have the permssions to download the attachments of this post.

3

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

I will verify this issue later and let you know the result.
Thanks very much for your feedback and sorry about the delay.

4

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

robynhub wrote:

1) When i list internal moderators i can't see the checkbox near the address so i can't' remove it.

Works for me here with both v1.5.0 and v1.6.0.
May i know which web browser do you use?

robynhub wrote:

2) I can't add internal members from Mail lIst profile page but if i add them by user profile it works (and i see the checkbox near the name)

BUG in iRedAdmin-Pro, both in v1.5.0 and v1.6.0. Patch attached (about line 353):

diff -r 1dbcd78614d7 libs/ldaplib/maillist.py
--- libs/ldaplib/maillist.py    Wed Jun 22 13:03:13 2011 +0800
+++ libs/ldaplib/maillist.py    Wed Jun 22 16:27:14 2011 +0800
@@ -350,7 +350,7 @@
         connutils = connUtils.Utils()
 
         # Get domain name and alias domains.
-        availableDomainNames = connutils.getAvailableDomainNames(domain)
+        availableDomainNames = connutils.getAvailableDomainNames(domain=domain)
         assignedGroups = [listname + '@' + v for v in availableDomainNames[1]]
 
         for member in members:
robynhub wrote:

3) when i delete a mail list when I see the user profile of an ex-member I see something like "Warning::Non Existent account" (see attach)

Fixed with above patch. But you have to remove these non-exist mail list in user profile page manually.
Sorry about this trouble.

5

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

ZhangHuangbin wrote:

Works for me here with both v1.5.0 and v1.6.0.
May i know which web browser do you use?

Firefox 5 (but even with 4) (not works)
Chrome 12.0.742.100 (not works)
Safari 5.0.5 (6533.21.1) (works)
All for mac.

Everything else is fixed. Thanks!

6

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

robynhub wrote:

Firefox 5 (but even with 4) (not works)
Chrome 12.0.742.100 (not works)
Safari 5.0.5 (6533.21.1) (works)
All for mac.

Fixed. There's an unclosed <td> tag. Patch attached:

diff -r 4f06fced8bd1 templates/default/ldap/maillist/profile.html
--- templates/default/ldap/maillist/profile.html    Wed Jun 22 16:36:18 2011 +0800
+++ templates/default/ldap/maillist/profile.html    Wed Jun 22 18:04:14 2011 +0800
@@ -288,7 +288,7 @@
                                         {% endif %}
 
                                         <tr class="{{ loop.cycle('odd', 'even') }} {{ mAccountStatus |lower }}">
-                                            <td class="checkbox"<input type="checkbox" name="mail" value="{{mail}}" class="checkbox" /></td>
+                                            <td class="checkbox"><input type="checkbox" name="mail" value="{{mail}}" class="checkbox" /></td>
                                             <td>{{cn}}
                                                 <a href="{{ctx.homepath}}/profile/user/general/{{mail}}" target="_blank"><img src="{{ctx.homepath}}/static/{{skin}}/images/action_edit.png" title="{{ _('Edit account profile') }}" class="fr-space"/></a>
                                                 {{ set_account_status_img(mAccountStatus) }}

You can search '<td class="checkbox"<' in file "templates/default/ldap/maillist/profile", then add a '>' before '<'.

7

Re: [SOLVED] [BUG] iRedAdmin-Pro various Mail List bugs

ZhangHuangbin wrote:

<td class="checkbox

Bulleye! It works!

Thanks a lot!