[intranet/modules/mesMachines/main.py] Fix du bug "on peut plus modifier ses machines"

Ignore-this: 66edbee00876be8be01d3ff53de3695

J'ai cru comprendre que ça avait un rapport avec la modif LDAP du @crans.org.
Même si je suis pas convaincu, "comme ça ça marche"

(Et puis bon, "raise Exception" c'était moche)

darcs-hash:20121205234643-2c9c1-c531c90a2b253798c48c356a12fad9eaee3cbc73.gz
This commit is contained in:
Vincent Le Gallic 2012-12-06 00:46:43 +01:00
parent d4453122d4
commit fc4cd9e2d3

View file

@ -25,6 +25,10 @@ else:
from ClassesIntranet.ModuleBase import ModuleBase from ClassesIntranet.ModuleBase import ModuleBase
class NotYourStuff(Exception):
"""Erreur levée si l'adhérent essaye de modifier une machine qu'il n'est pas une des siennes"""
pass
class main(ModuleBase): class main(ModuleBase):
_droits = ["personnel"] _droits = ["personnel"]
def title(self): def title(self):
@ -59,8 +63,12 @@ class main(ModuleBase):
def AJAXMachineInfo(self, mid): def AJAXMachineInfo(self, mid):
try: try:
machine = cherrypy.session['LDAP'].search('mid=' + mid)['machine'][0] machine = cherrypy.session['LDAP'].search('mid=' + mid)['machine'][0]
if machine.proprietaire().mail() != cherrypy.session['uid']: try:
raise Exception compte_machine = machine.proprietaire().compte()
except:
raise NotYourStuff("Il semble que vous cherchiez à modifier une machine dont le propriétaire n'a pas de compte Cr@ns…")
if compte_machine != cherrypy.session['uid']:
raise NotYourStuff("Vous ne pouvez pas modifier cette machine")
# zamok -> pour tester l'affichage des ports, des alias # zamok -> pour tester l'affichage des ports, des alias
#machine = cherrypy.session['LDAP'].search('mid=896')['machine'][0] #machine = cherrypy.session['LDAP'].search('mid=896')['machine'][0]
machineInfos = {} machineInfos = {}