You are viewing a plain text version of this content. The canonical link for it is here.

Posted to infrastructure-issues@apache.org by "Henri Yandell (Created) (JIRA)" <ji...@apache.org> on 2011/12/21 07:03:30 UTC

[jira] [Created] (INFRA-4238) Note list name on mail detail page

Note list name on mail detail page
----------------------------------

                 Key: INFRA-4238
                 URL: https://issues.apache.org/jira/browse/INFRA-4238
             Project: Infrastructure
          Issue Type: Improvement
      Security Level: public (Regular issues)
            Reporter: Henri Yandell
            Priority: Minor


Looking at http://mail-archives.apache.org/mod_mbox/www-infrastructure-dev/201112.mbox/%3CA603FFCE-623B-43E9-87F8-39BAA51C72D1@gbiv.com%3E - it would be nice to know what mailing list the email is to in the UI. Currently you have to infer it from links or the url.

Having it as some kind of title would be good. Possibly it could replace the 'list index' text for the link to the emails for that mailing list.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

Re: [PATCH] mod_mbox: show list name in the

Posted by Daniel Shahaf <d....@daniel.shahaf.name>.

Stefan Fritsch wrote on Mon, Jan 02, 2012 at 20:07:30 +0100:
> On Monday 02 January 2012, Daniel Shahaf wrote:
> > Henri Yandell (Created) (JIRA) wrote on Wed, Dec 21, 2011 at 
> 06:03:30 +0000:
> > > Note list name on mail detail page
> > > ----------------------------------
> > > 
> > >                  Key: INFRA-4238
> > >                  URL:
> > >                  https://issues.apache.org/jira/browse/INFRA-423
> > >                  8
> > >              
> > >              Project: Infrastructure
> > >           
> > >           Issue Type: Improvement
> > >       
> > >       Security Level: public (Regular issues)
> > >       
> > >             Reporter: Henri Yandell
> > >             Priority: Minor
> > > 
> > > Looking at
> > > http://mail-archives.apache.org/mod_mbox/www-infrastructure-dev/2
> > > 01112.mbox/%3CA603FFCE-623B-43E9-87F8-39BAA51C72D1@gbiv.com%3E -
> > > it would be nice to know what mailing list the email is to in
> > > the UI. Currently you have to infer it from links or the url.
> > > 
> > > Having it as some kind of title would be good. Possibly it could
> > > replace the 'list index' text for the link to the emails for that
> > > mailing list.
> > 
> > [[[
> > Tweak <h1/> title in the message display screen, for INFRA-4238.
> > 
> > Patch by: danielsh
> > 
> > * module-2.0/mod_mbox_out.c
> >   (mbox_ajax_browser, mbox_static_browser):
> >     Include the directory's basename in the title.
> > ]]]
> 
> Hi Daniel,
> 
> I have done some work on mod_mbox last week which I have committed 
> now. It includes displaying the list name in <h1> and <title> tags, 
> though implemented slightly differently than your patch. This and some 
> more code cleanup is now in trunk. This code should still work with 
> 2.2
> 

Saw the commits --- thanks.  I'll deploy the code on eos and let this
list know when that's done.

> In the new convert-charsets branch, I have committed some work to 
> actually convert mail bodies, subject and from lines to UTF-8. This 
> should take care of a load of displaying problems with various mails.

Sounds good.

> I have tested it with Firefox 9, Chrome, and KDE Konqueror 4.6.
> There are two caveats, though: It adds a field to the cache files. 
> This means that all mbox cache files need to be re-created. And it 
> depends on HTTPD 2.3.x functionality.
> 

Meaning that we can't deploy it for the private mail archives yet.

> It would of course be very nice if you could test the code from the 
> branch, but doing the re-indexing is of course more work. Maybe you 
> could pull some tricks with ZFS snapshots to allow for a fall-back 
> path.
> 

Okay.  I'll first see about installing trunk@HEAD, and later separately
about giving the branch a spin too.  Filed this as
https://issues.apache.org/jira/browse/INFRA-4273.

> Cheers,
> Stefan

Thanks,

Daniel

Re: [PATCH] mod_mbox: show list name in the

Posted by Stefan Fritsch <sf...@sfritsch.de>.

On Monday 02 January 2012, Daniel Shahaf wrote:
> Henri Yandell (Created) (JIRA) wrote on Wed, Dec 21, 2011 at 
06:03:30 +0000:
> > Note list name on mail detail page
> > ----------------------------------
> > 
> >                  Key: INFRA-4238
> >                  URL:
> >                  https://issues.apache.org/jira/browse/INFRA-423
> >                  8
> >              
> >              Project: Infrastructure
> >           
> >           Issue Type: Improvement
> >       
> >       Security Level: public (Regular issues)
> >       
> >             Reporter: Henri Yandell
> >             Priority: Minor
> > 
> > Looking at
> > http://mail-archives.apache.org/mod_mbox/www-infrastructure-dev/2
> > 01112.mbox/%3CA603FFCE-623B-43E9-87F8-39BAA51C72D1@gbiv.com%3E -
> > it would be nice to know what mailing list the email is to in
> > the UI. Currently you have to infer it from links or the url.
> > 
> > Having it as some kind of title would be good. Possibly it could
> > replace the 'list index' text for the link to the emails for that
> > mailing list.
> 
> [[[
> Tweak <h1/> title in the message display screen, for INFRA-4238.
> 
> Patch by: danielsh
> 
> * module-2.0/mod_mbox_out.c
>   (mbox_ajax_browser, mbox_static_browser):
>     Include the directory's basename in the title.
> ]]]

Hi Daniel,

I have done some work on mod_mbox last week which I have committed 
now. It includes displaying the list name in <h1> and <title> tags, 
though implemented slightly differently than your patch. This and some 
more code cleanup is now in trunk. This code should still work with 
2.2

In the new convert-charsets branch, I have committed some work to 
actually convert mail bodies, subject and from lines to UTF-8. This 
should take care of a load of displaying problems with various mails.
I have tested it with Firefox 9, Chrome, and KDE Konqueror 4.6.
There are two caveats, though: It adds a field to the cache files. 
This means that all mbox cache files need to be re-created. And it 
depends on HTTPD 2.3.x functionality.

It would of course be very nice if you could test the code from the 
branch, but doing the re-indexing is of course more work. Maybe you 
could pull some tricks with ZFS snapshots to allow for a fall-back 
path.

Cheers,
Stefan

[PATCH] mod_mbox: show list name in the

Posted by Daniel Shahaf <d....@daniel.shahaf.name>.

Henri Yandell (Created) (JIRA) wrote on Wed, Dec 21, 2011 at 06:03:30 +0000:
> Note list name on mail detail page
> ----------------------------------
> 
>                  Key: INFRA-4238
>                  URL: https://issues.apache.org/jira/browse/INFRA-4238
>              Project: Infrastructure
>           Issue Type: Improvement
>       Security Level: public (Regular issues)
>             Reporter: Henri Yandell
>             Priority: Minor
> 
> 
> Looking at
> http://mail-archives.apache.org/mod_mbox/www-infrastructure-dev/201112.mbox/%3CA603FFCE-623B-43E9-87F8-39BAA51C72D1@gbiv.com%3E
> - it would be nice to know what mailing list the email is to in the
> UI. Currently you have to infer it from links or the url.
> 
> Having it as some kind of title would be good. Possibly it could
> replace the 'list index' text for the link to the emails for that
> mailing list.
> 

[[[
Tweak <h1/> title in the message display screen, for INFRA-4238.

Patch by: danielsh

* module-2.0/mod_mbox_out.c
  (mbox_ajax_browser, mbox_static_browser):
    Include the directory's basename in the title.
]]]

[[[
Index: module-2.0/mod_mbox_out.c
===================================================================
--- module-2.0/mod_mbox_out.c   (revision 1188408)
+++ module-2.0/mod_mbox_out.c   (working copy)
@@ -17,6 +17,8 @@
 /* This file contains all output functions.
  */
 
+#include <libgen.h> /* basename() */
+
 #include "mod_mbox.h"
 
 #ifdef APLOG_USE_MODULE
@@ -886,9 +888,11 @@ apr_status_t mbox_ajax_browser(request_rec *r)
 {
     mbox_dir_cfg_t *conf;
     char *baseURI;
+    const char *base_path;
 
     conf = ap_get_module_config(r->per_dir_config, &mbox_module);
     baseURI = get_base_uri(r);
+    base_path = get_base_path(r);
 
     ap_set_content_type(r, "text/html; charset=utf-8");
 
@@ -920,7 +924,8 @@ apr_status_t mbox_ajax_browser(request_rec *r)
     ap_rprintf(r,
                " <body id=\"archives\" onload=\"javascript:loadBrowser ('%s');\">\n",
                baseURI);
-    ap_rputs("  <h1>Mailing list archives</h1>\n\n", r);
+    ap_rprintf(r, "  <h1>%s mailing list archives</h1>\n\n",
+                  basename(base_path));
 
     ap_rputs("  <h5>\n", r);
 
@@ -930,7 +935,7 @@ apr_status_t mbox_ajax_browser(request_rec *r)
     }
 
     ap_rprintf(r, "<a href=\"%s\" title=\"Back to the list index\">"
-               "List index</a></h5>", get_base_path(r));
+               "List index</a></h5>", base_path);
 
     /* Output a small notice if no MboxScriptPath configuration
        directive was specified. */
@@ -1122,6 +1127,7 @@ int mbox_static_message(request_rec *r, apr_file_t
     Message *m;
 
     char *baseURI, *from, **context, *msgID, *escaped_msgID;
+    const char *base_path = get_base_path(r);
 
     conf = ap_get_module_config(r->per_dir_config, &mbox_module);
     baseURI = get_base_uri(r);
@@ -1166,7 +1172,8 @@ int mbox_static_message(request_rec *r, apr_file_t
 
     ap_rputs(" </head>\n\n", r);
     ap_rputs(" <body id=\"archives\">\n", r);
-    ap_rputs("  <h1>Mailing list archives</h1>\n\n", r);
+    ap_rprintf(r, "  <h1>%s mailing list archives</h1>\n\n",
+                  basename(base_path));
 
     ap_rputs("  <h5>\n", r);
 
@@ -1176,7 +1183,7 @@ int mbox_static_message(request_rec *r, apr_file_t
     }
 
     ap_rprintf(r, "<a href=\"%s\" title=\"Back to the list index\">"
-               "List index</a></h5>", get_base_path(r));
+               "List index</a></h5>", base_path);
 
     /* Display context message list */
     from = ESCAPE_OR_BLANK(r->pool, m->from);
]]]

> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> For more information on JIRA, see: http://www.atlassian.com/software/jira
> 
>