Problema di permessi su alcune operazioni sugli ordini fatte dai referenti del Patto relativo

[matteo88]

Se un Patto ha più referenti e uno di questi apre un ordine, gli altri referenti non possono decurtare i membri del GAS che hanno ordinato nè registrare la fattura dell'ordine.

[matteo88]

A livello di codice, il problema ricade in questa riga:

if not self.__loggedusr.has_perm(CASH, obj=ObjectWithContext(self.__order.gas)) and \
    not self.__loggedusr == self.__order.referrer_person.user:
        raise PermissionDenied(ugettext("You are not a cash_referrer or the order's referrer, you cannot update GASMembers cash!"))

in particolare, il controllo

not self.__loggedusr == self.__order.referrer_person.user:

vuole indicare che solo il referente che ha aperto l'ordine può eseguire le relative operazioni su di esso.

In realtà, questa è una svista perchè in un Patto potrebbero esserci più referenti che in simbiosi dovrebbero poter gestire le operazioni, ad esempio uno di essi apre l'ordine ed un altro esegue le operazioni di registrazione e decurtazioni. Questo è utile soprattutto su ordini più complessi che necessitano un impegno temporale ed organizzativo non indifferente, che dovrebbe essere diviso tra più persone. Inoltre è utile anche per favorire un'ottica di coinvolgimento tra i soci del GAS che sono anche referenti (o che potrebbero diventarlo).

La soluzione a questo problema consiste nel modificare il codice (ove necessario) in modo che a tutti i referenti del Patto siano concessi i permessi per modificare gli ordini, quindi da:

if not self.__loggedusr.has_perm(CASH, obj=ObjectWithContext(self.__order.gas)) and \
    not self.__loggedusr == self.__order.referrer_person.user:
        raise PermissionDenied(ugettext("You are not a cash_referrer or the order's referrer, you cannot update GASMembers cash!"))

a

if not self.__loggedusr.has_perm(CASH, obj=ObjectWithContext(self.__order.gas)) and \
    not self.__loggedusr == self.__order.referrers:
        raise PermissionDenied(ugettext("You are not a cash_referrer or the order's referrer, you cannot update GASMembers cash!"))

[feroda]

@matteo88 In specifications there are referrers for deliveries, but I agree that should be easier to allow every pact referrer to curtail gasmembers of related orders.

Imho It should be done