[trunk - base_calendar] exrule field without functionality [IMP]

Bug #530681 reported by Omar (Pexego)
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Confirmed
Wishlist
OpenERP R&D Addons Team 1

Bug Description

Hi,

I'm interesting in base_calendar module, it's great. I found some bugs.
For example the field exrule isn't implemented, this field is very useful because in dateutil rules you can set exceptions with a rule. I'm implemented this functionality with this field. I attached an improvement. Please take a look to file and maybe can improve my source, but the functionality it's ok and it's works.

If I set exrule fields to: FREQ=WEEKLY;BYDAY=SA,SU;INTERVAL=1;UNTIL=20040326220000Z I avoid weekends in recursion set for example FREQ=DAILY;INTERVAL=1;UNTIL=20040326235959Z start in 2004-03-15 19:00:00.

Maybe could create a wizard to set exrule field as custom rule wizard.

Then in my attachment I fixed a bug in compute_rule_string() if freq was None, because res_obj and obj isn't defined and the functionality is in function caller.

Tags: pexego
summary: - [base_calendar] exrule field without functionality [IMP]
+ [trunk - base_calendar] exrule field without functionality [IMP]
Changed in openobject-addons:
importance: Undecided → Wishlist
assignee: nobody → rpa (Open ERP) (rpa-openerp)
Changed in openobject-addons:
status: New → In Progress
Revision history for this message
Omar (Pexego) (omar7r) wrote :

Hi rpa,

I reattached the file with new improvements about read and search methods, please review the file because with large quantity of registries it only show in calendar view the 100 or default limit registries by date not filtered, now it uses the dates and recursion params calculating visible registries and then it was very slow in read because it was reading for each id and virtual id in read again all same data, now first read all data and then maps it and creates virtual ids.

Thanks

Revision history for this message
Omar (Pexego) (omar7r) wrote :
Revision history for this message
Omar (Pexego) (omar7r) wrote :

Hi rpa,

I see an error in my improvement that it doesn't allow to search for registries with rrule defined then i reattached the patch,

Please take a look to patch, the read method is very improved, it is very much faster.

Now, in last revision my patch can't be applied with bzr patch.

Thanks

Changed in openobject-addons:
status: In Progress → Confirmed
Changed in openobject-addons:
assignee: rpa (Open ERP) (rpa-openerp) → nobody
tfr (Openerp) (tfr)
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 1 (openerp-dev-addons1)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.