Merge lp:~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto into lp:addons-vauxoo/7.0

[hbto [Vauxoo] http://www.vauxoo.com]

This MP is intended to add a field to the model
user.story in order to be able to provide end users
with better reports.

[Moisés López - http://www.vauxoo.com (moylop260)]

@Hbto,
FYI Esta MP está ciclando uno de los test, tengan en cuenta esto antes de mergear.
http://runbot.vauxoo.com/Chuck_Norris.html#erp_vauxoo_com_-mp-218128

El log es el siguiente
Module project_timesheet: loading test/worktask_entry_to_timesheetline_entry.yml
 Create a user 'HR Manager'
 Create a product with type service used to specify employees designation
 Create an analytic journal for employees timesheet
 Create an employee 'HR Manager' for user 'HR Manager'
 Create a timesheet sheet for HR manager
 Create a project 'Timesheet Management'
 Create a task 'Get all timesheet records'
 Open the task
 Make a work task entry 'Get work calendar of all employees' of 10 hours done by HR manager.
En esta última línea se queda pegado, y runbot detecta que ya lleva 40 minutos generándose la instancia, y la frena, etiquetándola como "too long".

@Rafa,
Para aprobar una MP, te recomiendo primero probar en runbot en
http://runbot.vauxoo.com/Chuck_Norris.html
Busca el número de la propuesta de merge, y le das "force build", una vez que se genera, puedes conectarte con usuario admin y password admin, para probar funcionalmente la vaina en desarrollo.
Si no está en verde o no te satisface, dispara alarmas, please.
Si todo trabaja fino funcionalmente, ahora sí apruebas.

review: Needs Fixing

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

Moy no se ha mezclado y no se ha puesto a producción por esa precisa razón,

Si estuviera en producción tu preocupación tendría toda la razón, evita el
proceso de minority report
On May 4, 2014 7:32 PM, "Moisés López - http://www.vauxoo.com" <
<email address hidden>> wrote:

> Review: Needs Fixing
>
> @Hbto,
> FYI Esta MP está ciclando uno de los test, tengan en cuenta esto antes de
> mergear.
> http://runbot.vauxoo.com/Chuck_Norris.html#erp_vauxoo_com_-mp-218128
>
> El log es el siguiente
> Module project_timesheet: loading
> test/worktask_entry_to_timesheetline_entry.yml
> Create a user 'HR Manager'
> Create a product with type service used to specify employees designation
> Create an analytic journal for employees timesheet
> Create an employee 'HR Manager' for user 'HR Manager'
> Create a timesheet sheet for HR manager
> Create a project 'Timesheet Management'
> Create a task 'Get all timesheet records'
> Open the task
> Make a work task entry 'Get work calendar of all employees' of 10 hours
> done by HR manager.
> En esta última línea se queda pegado, y runbot detecta que ya lleva 40
> minutos generándose la instancia, y la frena, etiquetándola como "too long".
>
> @Rafa,
> Para aprobar una MP, te recomiendo primero probar en runbot en
> http://runbot.vauxoo.com/Chuck_Norris.html
> Busca el número de la propuesta de merge, y le das "force build", una vez
> que se genera, puedes conectarte con usuario admin y password admin, para
> probar funcionalmente la vaina en desarrollo.
> Si no está en verde o no te satisface, dispara alarmas, please.
> Si todo trabaja fino funcionalmente, ahora sí apruebas.
>
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto/+merge/218128
> You proposed
> lp:~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto for
> merging.
>

[Moisés López - http://www.vauxoo.com (moylop260)]

Excelente!
Evita tener una MP con estado como aprobada, cuando aun está en WIP, please.

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

Copiado
On May 4, 2014 8:12 PM, "Moisés López - http://www.vauxoo.com" <
<email address hidden>> wrote:

> Excelente!
> Evita tener una MP con estado como aprobada, cuando aun está en WIP,
> please.
>
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto/+merge/218128
> You proposed
> lp:~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto for
> merging.
>

[lp:~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto]

1010. By hbto [Vauxoo] http://www.vauxoo.com on 2014-05-05

[FIX] Stupid Level-8 Mistake

[Moisés López - http://www.vauxoo.com (moylop260)]

Que coraje, cuando es un pdb el que te estaba jodiendo la vaina.

Rafa,
Si quieres probarlo en desarrollo, esta es la instancia.
http://runbot.vauxoo.com/Chuck_Norris.html#erp_vauxoo_com_-mp-218128

¡Saludos!

=== modified file 'user_story/model/user_story.py'
--- user_story/model/user_story.py	2014-04-25 23:32:52 +0000
+++ user_story/model/user_story.py	2014-05-05 13:19:17 +0000
@@ -155,6 +155,37 @@
 
         return False
      
+    def _hours_get(self, cr, uid, ids, field_names, args, context=None):
+        res = {}
+        cr.execute('''
+            SELECT us.id, COALESCE(SUM(ptw.hours))
+            FROM project_task_work ptw
+            INNER JOIN project_task pt ON pt.id = ptw.task_id
+            INNER JOIN user_story us ON us.id = pt.userstory_id
+            WHERE us.id IN %s 
+            GROUP BY us.id
+        ''',(tuple(ids),))
+        hours = dict(cr.fetchall())
+        for us_brw in self.browse(cr, uid, ids, context=context):
+            res[us_brw.id] = hours.get(us_brw.id, 0.0)
+        return res
+
+    def _get_user_story_from_ptw(self, cr, uid, ids, context=None):
+        result = {}
+        task_ids = {}
+        for work in self.pool.get('project.task.work').browse(cr, uid, ids, context=context):
+            if work.task_id: result[work.task_id.id] = True
+        task_ids = task_ids.keys()
+        for task in self.pool.get('project.task').browse(cr, uid, task_ids, context=context):
+            if task.userstory_id: result[task.userstory_id.id] = True
+        return result.keys()
+
+    def _get_user_story_from_pt(self, cr, uid, ids, context=None):
+        result = {}
+        for task in self.pool.get('project.task').browse(cr, uid, ids, context=context):
+            if task.userstory_id: result[task.userstory_id.id] = True
+        return result.keys()
+
     _columns = {
         'name': fields.char('Title', size=255, required=True, readonly=False, translate=True),
         'owner': fields.char('Owner', size=255, readonly=False),
@@ -193,7 +224,12 @@
         'categ_ids': fields.many2many('project.category','project_category_user_story_rel','userstory_id','categ_id', string="Tags"),
         'implementation': fields.text('Implementation Conclusions', translate=True),
         'help': fields.boolean('Show Help', help='Allows you to show the help in the form'),
-
+        'effective_hours': fields.function(_hours_get, string='Hours Spent', help="Computed using the sum of the task work done.",
+            store = {
+                _name: (lambda s, c, u, ids, cx={}: ids, ['task_ids'], 10),
+                'project.task': (_get_user_story_from_pt, ['work_ids'], 10),
+                'project.task.work': (_get_user_story_from_ptw, ['hours'], 10),
+            }),
     }
     _defaults = {
         'name': lambda *a: None,

=== modified file 'user_story/view/userstory_view.xml'
--- user_story/view/userstory_view.xml	2014-04-25 17:45:49 +0000
+++ user_story/view/userstory_view.xml	2014-05-05 13:19:17 +0000
@@ -47,6 +47,7 @@
 				</group>
 				<group>
 					<field name="planned_hours" />
+					<field name="effective_hours" />
 					<field name="sk_id" context="{'current_model3': 'sprint.kanban'}" />
 					<field name="priority_level" widget="selection"/>
                     <field name="categ_ids" widget="many2many_tags" context="{'current_model1': 'project.category'}"/>
@@ -144,6 +145,7 @@
           <field name="code" />
           <field name="priority_level" />
           <field name="planned_hours" sum="Total hours"/>
+          <field name="effective_hours" sum="Total hours"/>
           <field name="project_id" />
           <field name="user_id" />
           <field name="sk_id" select="True" />