[Building Sakai] Installing Australe 2.7.0.x on Sakai 2.7.1

[Rashmi Maheshwari]

Hi Warwick Chapman and Jean-Francois,
Melete has defined its own 2 roles - melete.author (basically with 
maintain role and we call this person as INSTRUCTOR) and melete.student 
(access role and similarly we call this person STUDENT). We have fixed 
this issue in our upcoming release. Take a look at rev 69255 and rev 69283.

thanks,
Rashmi

=================================
Here are rev details. These revisions apply changes to few new classes 
and methods, so be very careful if you are planning to apply the patch.


Revision: 69255
Author: mallika at etudes.org
Date: 4:48:27 PM, Thursday, July 15, 2010
Message:
Fix for ME-1249, this fix replaces sessionmap.get(role) with calls to 
meletesecurityservice. It consolidates student instructor view calls to 
listviewAction, removes student preview image from what's next page, 
does the role check at db level for getnextseq and getprevseq, removes 
some unnecessary code with sequence numbers at the bean layer.
----
Modified : 
/etudes/melete/trunk/melete-api/src/java/org/etudes/api/app/melete/ModuleService.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/BookmarkPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ListModulesPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/MeleteSiteAndUserInfo.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/NavPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ViewModulesPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ViewNextStepsPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ViewSectionsPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/webapp/WEB-INF/faces-config.xml
Modified : 
/etudes/melete/trunk/melete-app/src/webapp/melete/view_whats_next.jsp
Modified : 
/etudes/melete/trunk/melete-impl/src/java/org/etudes/component/app/melete/ModuleDB.java
Modified : 
/etudes/melete/trunk/melete-impl/src/java/org/etudes/component/app/melete/ModuleServiceImpl.java

Revision: 69283
Author: mallika at etudes.org
Date: 2:16:45 PM, Monday, July 19, 2010
Message:
Fix for ME-1249, removing isauthor and isstudent check from constructor. 
Removing import statement for facesmessage from viewmodulespage as it is 
not needed
----
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ListModulesPage.java
Modified : 
/etudes/melete/trunk/melete-app/src/java/org/etudes/tool/melete/ViewModulesPage.java

==========================




Warwick Chapman wrote:
> J-F
>
> As per the attached image, the role is "access".
>
> Is that right?
>
> -- Warwick Bruce Chapman | +27 83 7797 094 | http://warwickchapman.com
>
>
> On Mon, Sep 20, 2010 at 4:34 PM, Jean-Francois Leveque 
> <jean-francois.leveque at upmc.fr <mailto:jean-francois.leveque at upmc.fr>> 
> wrote:
>
>     Is seems role is null on line 67 of
>     melete/melete-app/src/java/org/etudes/tool/melete/NavPage.java.
>
>     Looks like this part of the code should be made more resilient.
>
>     Please try to replace
>     if (role.equals("INSTRUCTOR"))
>     with
>     if ("INSTRUCTOR".equals(role))
>
>     It nonetheless seems your unprivileged user has no role, which
>     could be a important problem.
>
>
>     Cheers,
>
>     J-F
>
>
> ------------------------------------------------------------------------
>


-- 
Rashmi Maheshwari
Learning Systems Developer
Etudes Inc
http://etudes.org

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20100920/4c74be66/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/jpeg
Size: 62281 bytes
Desc: not available
Url : http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20100920/4c74be66/attachment.jpe