Create alternate route for users that cannot force assign.

shipment.py

 #This file is part of Tryton.  The COPYRIGHT file at the top level of
 #this repository contains the full copyright notices and license terms.
 "Shipment"
 from trytond.model import ModelWorkflow, ModelView, ModelSQL, fields
 from trytond.modules.company import CompanyReport
 from trytond.wizard import Wizard
 from trytond.backend import TableHandler
 
 STATES = {
     'readonly': "state in ('cancel', 'done')",
@@ skipping 1239 matching lines @@
 class AssignShipmentOut(Wizard):
     'Assign Shipment Out'
     _name = 'stock.shipment.out.assign'
     states = {
         'init': {
             'result': {
                 'type': 'choice',
                 'next_state': '_choice',
             },
         },
+        'notify': {

[ced, 2010/01/15 23:23:25]

I think it should be named, assign_failed

+            'actions': ['_moves'],
+            'result': {
+                'type': 'form',
+                'object': 'stock.shipment.out.assign.ask_force',

[ced, 2010/01/15 23:23:25]

You must use an other object

+                'state': [
+                    ('end', 'Ok', 'tryton-ok', True),
+                ],
+            },
+        },

[Ian Wilson, 2010/01/15 19:27:33]

There seemed no way to just hide the button in a dialog so I created an
alternate path to notify users without the ability to force assign that
assigning failed.

[udono, 2010/01/15 20:08:06]

I think its the right way you go.

         'ask_force': {
             'actions': ['_moves'],
             'result': {
                 'type': 'form',
                 'object': 'stock.shipment.out.assign.ask_force',
                 'state': [
                     ('force', 'Force Assign', 'tryton-go-next'),
                     ('end', 'Ok', 'tryton-ok', True),
                 ],
             },
         },
         'force': {
             'result': {
                 'type': 'action',
                 'action': '_force',
                 'state': 'end',
             },
         },
     }
 
     def _choice(self, cursor, user, data, context=None):
         shipment_out_obj = self.pool.get('stock.shipment.out')
+        user_obj = self.pool.get('res.user')
 
         shipment_out_obj.workflow_trigger_validate(cursor, user, data['id'],
                 'assign', context=context)
         shipment = shipment_out_obj.browse(cursor, user, data['id'],
                 context=context)
         if not [x.id for x in shipment.inventory_moves if x.state == 'draft']:
             return 'end'
         else:
-            return 'ask_force'
+            users = user_obj.browse(cursor, user, [user], context=context)

[ced, 2010/01/15 23:23:25]

It will be better to search on group with users = user and id = group id.

+            if users:
+                for group in users[0].groups:
+                    if group.name == 'Stock Force Assignment':

[Ian Wilson, 2010/01/15 19:27:33]

Is there a better way to check if a user belongs to a group specified in
XML(stock.xml) ?

[udono, 2010/01/15 20:08:06]

I don't know a better way.

[ced, 2010/01/15 23:23:25]

You must use ir.model.data to get the id.
But I think it will be best to read the group id from the workflow.transition.

[Ian Wilson, 2010/01/16 02:39:39]

I think I understand all the comments except for this one.  What do you mean by
"read the group id from the workflow.transition"?  From
shipmentinternal_trans_waiting_assigned_force ?

[ced, 2010/01/16 09:12:33]

Yes but don't forget that there is also a wizard stock.shipment.internal.assign

+                        return 'ask_force'
+            return 'notify'
 
     def _moves(self, cursor, user, data, context=None):
         shipment_out_obj = self.pool.get('stock.shipment.out')
         shipment = shipment_out_obj.browse(cursor, user, data['id'],
                 context=context)
         return {'inventory_moves': [x.id for x in shipment.inventory_moves
             if x.state == 'draft']}
 
     def _force(self, cursor, user, data, context=None):
         shipment_out_obj = self.pool.get('stock.shipment.out')
@@ skipping 588 matching lines @@
                 'to_location_ids' : to_location_ids}
 
 
     def get_compare_key(self, move, compare_context):
         from_location_ids = compare_context['from_location_ids']
         to_location_ids = compare_context['to_location_ids']
         return [from_location_ids.index(move.from_location.id),
                 to_location_ids.index(move.to_location.id)]
 
 InteralShipmentReport()