[ 
https://jira.nuxeo.org/browse/NXP-4950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=72105#action_72105
 ] 

Christophe Capon commented on NXP-4950:
---------------------------------------

OK, I had some time to take a look at your remarks, Tiry, thanks again.

* notifications-core : see attached file nuxeo-platform-notification-core.diff, 
which is a diff from the 5.3.0. I am quite familiar with publish/subscribe 
architectures, if you start refactoring the notification service, I could help 
when the alert management is up and running...

* ecm:lock : oops. Forgot that one. Yes, there was a small change in 
NXQLQueryMaker. See NXQLQueryMaker.java.diff. I saw a post on the jiveforums 
mailine list about that one, btw.

* Packaging : DeferredAlertManagementServiceImplComponent moved. I had the same 
concern with AlertManagementServiceImplComponent, which is at the moment in the 
nuxeo-platform-alert-api. I chose to put in the api everything a client might 
need (no big breakthrough here :-), but also everything for s/one who wants to 
provide extension points. The problem is that the alerts-framework.xml has to 
reference this component and the extension points at the same time. If I put 
the component in the -core, I have a dependence cycle between api and core. The 
alternative is to put the service extension point definition in the -core along 
with AlertManagementServiceImplComponent, but the -api would be only a client 
api. Philosophical problem here, let me know the way you look at this.

* TasksProvider : the problem is that jbpm wants a list of users with 
getCurrentTaskInstances. I want the tasks for **all** users, and it does not 
seem possible. I tried to avoid Hibernate, but unsuccessfully. Maybe I missed 
sth, but when I googled this I found no appropriate answer.

* Error Management : yeah, that eclipse's fault, along with all the bugs and 
poor formatting :-) No really, I removed the printStackTrace, and some other 
exceptions which were no longer used.

* DocumentModelAdapter : I'll try, but it seems too generic for me. The idea 
was to encapsulate the properties with classes to have strong type checking. I 
just browsed rapidly the DocumentModelAdapter; i'll take a closer look.

* UnrestrictedSession : now that the easy parts are done, I'll take a look this 
w-end. I need to look deeper in this session stuff.

* Code formatting : I d/l the nuxeo eclipse extensions and I'll give them a 
shot on all my files in a single commit soon.


> User settings management and Batch notifications
> ------------------------------------------------
>
>                 Key: NXP-4950
>                 URL: https://jira.nuxeo.org/browse/NXP-4950
>             Project: Nuxeo Enterprise Platform
>          Issue Type: New Feature
>    Affects Versions: 5.3 GA
>         Environment: Tested on ubuntu + tomcat + derby, ubuntu + jboss + 
> postgresql
>            Reporter: Christophe Capon
>         Attachments: AlertManagement.tar.gz
>
>
> Our nuxeo user start complaining about Nuxeo verbosity regarding 
> notifications. A sustained activity on the database may trigger a significant 
> amount of mails.
> To resolve this point, we have developed two services :
> 1) An alert manager service : it collects all the alerts worth being notified 
> to an user (locked documents, overdue tasks, subscriptions...). It groups all 
> those alerts in a single mail and sends it periodically to the user. This 
> service can be extended via extension points, if you want to add other alerts 
> (eg a document has reached its expiration date,....). At this point, we need 
> to allow the user to customize some parts of the alert mail.
> 2) There comes the second service. We want to allow the user to setup the 
> frequency of the alert mail, to select what he wants inside the mail (in our 
> development, locked documents and overdue tasks are sent no matter what the 
> user wants, but the user may want to have all his future tasks, or tasks 
> without target date). We created a UserSettings management service, allowing 
> the user to select his preferences. This service is similar to the Windows 
> Control Panel or Ubuntu Preferences Menu. So far, it only handles the alert 
> configuration, but you can add other settings managers through extension 
> points.
> A first stable but perfectible release is almost done, and I will keep you 
> posted when the code is available, with unit tests and hopefully selenium.
> Hopefully, this is the right place to post my code ?

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://jira.nuxeo.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
ECM-tickets mailing list
ECM-tickets@lists.nuxeo.com
http://lists.nuxeo.com/mailman/listinfo/ecm-tickets

Reply via email to