[Building Sakai] Questions about 2.8.1 site email address edit, multiple email aliases

Gast, Cynthia (cmw6s) cmw6s at eservices.virginia.edu
Thu May 17 08:04:39 PDT 2012


Hi Matthew:

Thanks for the references, very helpful.  SAK-20063 does look like the 2.8.1 code I just changed to only allow one site email alias to exist (as it was in our 2.7.1 user experience); I'll  look at possibly back-porting SAK-19831 to the 2.8.1 version we are about to deploy.

Here is what I changed (adding a property) to restrict email archive aliases to one per site, allowing users to change the alias.  In conjunction with this, we set "wsetup.disable.siteAlias=true" to suppress the display of the Site Info Site URL Alias edit capability.


--- uva-collab/mailarchive/branches/sakai_2-8-x_dev/mailarchive-tool/tool/src/java/org/sakaiproject/mailarchive/tool/MailboxAction.java 2012/05/16 09:53:29     17564
+++ uva-collab/mailarchive/branches/sakai_2-8-x_dev/mailarchive-tool/tool/src/java/org/sakaiproject/mailarchive/tool/MailboxAction.java 2012/05/16 20:23:10     17565
@@ -958,6 +958,17 @@
                                                if ( alias == null ) {
                                                        AliasService.removeTargetAliases(channel.getReference());
                                                } else {
+                                                       // UVa adds test for whether sites are restricted to
+                                                       // a single site email address.  UVa SAK-2747.
+                                                       Boolean singleSiteEmailAlias =
+                                                           ServerConfigurationService.getBoolean("mailarchive.singleSiteEmailAlias",
+                                                               Boolean.FALSE);
+
+                                                       if (singleSiteEmailAlias) {
+                                                               // Limit the site email to a single alias, so first
+                                                               // remove any existing aliases.
+                                                               AliasService.removeTargetAliases(channel.getReference());
+                                                       }
                                                        AliasService.setAlias(alias, channel.getReference());
                                                }
                                        }


Thanks,
Cindy @ UVa
cmw6s at virginia.edu


________________________________
From: Matthew Jones [matthew at longsight.com]
Sent: Wednesday, May 16, 2012 10:52 AM
To: Gast, Cynthia (cmw6s)
Cc: sakai-dev at collab.sakaiproject.org
Subject: Re: [Building Sakai] Questions about 2.8.1 site email address edit, multiple email aliases

Hi Cynthia, I think this was the jira around that

https://jira.sakaiproject.org/browse/SAK-20063

I believe that if you had a correct alias, and entered an incorrect alias, that the correct alias wasn't removed and wasn't reusable again so you were stuck.

Better error checking on the alias length in
https://jira.sakaiproject.org/browse/SAK-19831

Should make it so this can't happen anyway though.

On Wed, May 16, 2012 at 10:42 AM, Gast, Cynthia (cmw6s) <cmw6s at eservices.virginia.edu<mailto:cmw6s at eservices.virginia.edu>> wrote:
Following up on my own question... I see what changed and how to address this for our needs at UVa.

I found the "mailarchive" code in
    mailarchive-tool/tool/src/java/org/sakaiproject/mailarchive/tool/MailboxAction.java, doUpdate()

   changed slightly between 2.7.x and 2.8.x, such that now an edit to the site email address creates another email address instead of modifying the existing email address.

We will revert to the 2.7.x behavior because we want a single site email address, that users can change if desired.  Together with this change, we also hide the Site Info Site URL alias to avoid the creation of multiples.

Cheers,
Cindy @ UVa
cmw6s at virginia.edu<mailto:cmw6s at virginia.edu>


________________________________
From: sakai-dev-bounces at collab.sakaiproject.org<mailto:sakai-dev-bounces at collab.sakaiproject.org> [sakai-dev-bounces at collab.sakaiproject.org<mailto:sakai-dev-bounces at collab.sakaiproject.org>] on behalf of Gast, Cynthia (cmw6s) [cmw6s at eservices.virginia.edu<mailto:cmw6s at eservices.virginia.edu>]
Sent: Monday, May 14, 2012 10:32 AM
To: sakai-dev at collab.sakaiproject.org<mailto:sakai-dev at collab.sakaiproject.org>
Subject: [Building Sakai] Questions about 2.8.1 site email address edit, multiple email aliases

Hello:
I've been reviewing changes made between 2.7.x and 2.8.x related to both multiple site email aliases, and the ability to edit/delete them.  SAK-14910 deals with this.

UVa is preparing to roll out Sakai 2.8.1, and we desire the following behavior for the site email address:
    -- Only one site email address, that the user can edit

It seems changes have been made since Sakai 2.7.1, and wondering if this is something we now have to code around to get back to?  I've spent some time on this, not found a way short of code modification, but possible I missed something?

Question 1:  Is this single site email address behavior still possible in 2.8.1, perhaps with a property or combination of properties - or any patches I can apply?


In Sakai 2.7.1, UVa set the wsetup/siteInfo property to hide the Site URL Alias (wsetup.disable.siteAlias=true) , which was creating multiple site email addresses when the user attempted to edit the existing one (with no way to delete them); with this change in place our users could modify the single email address via the Email Archive tool (if desired).  So, that worked for us in 2.7.1.  Does not work now, as my attempt to modify the single existing email address in Email Archive/Options creates an additional site email address instead.  This UI behavior is unexpected; one would expect edit to change the existing string.

BTW, I did try setting the below property (SAK-14910) in our 2.8.1 instance (after also showing the Site Info Site URL field),  and it still produces another site email alias instead of allowing me to edit the one that exists, and did not let me delete newly created additional site email addresses:
    # Enable email alias edit/delete for existing sites
    site-manage.enable.alias.edit=true
... Wondering if anyone else sees this behavior in 2.8.1?  I can enter a JIRA about this, if needed?

Question 2: What is the current 2.8.x, 2.9.x design intent for site email alias?  Multiples per site, with the ability to edit existing, delete existing, add additional?


Thanks for information on this issue, especially if we can conditionally revert to the "single site email address + edit that one to change it" behavior.
Cindy @ UVa
    (fka Cynthia Walters)



_______________________________________________
sakai-dev mailing list
sakai-dev at collab.sakaiproject.org<mailto:sakai-dev at collab.sakaiproject.org>
http://collab.sakaiproject.org/mailman/listinfo/sakai-dev

TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org<mailto:sakai-dev-unsubscribe at collab.sakaiproject.org> with a subject of "unsubscribe"

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20120517/f2b51deb/attachment.html 


More information about the sakai-dev mailing list