search

tnc-ba / strongTNC

BYOD TNC Database Management Tool
GNU Affero General Public License v3.0
0 stars 0 forks source link

Deleting policies is working again #72

Closed d22 closed 10 years ago

d22 commented 10 years ago

Deleting policies was broken because of a changed variable name (missmatch with urls.py).

dbrgn commented 10 years ago

Thanks, nicely spotted, but the right thing to do here would be to rename the URL rules (all of them that contain camelCase), not the variable. Agree?

And we need more tests in the future to more quickly spot such problems :)

d22 commented 10 years ago

Yes, I do agree, but I changed the varibale, because I changed it before. I didn't want to update all urls, but just fix the broken one. I'd say we merge this to fix the bug, and update all urls/variables in the code-healt task. Oke?

dbrgn commented 10 years ago

:+1: gogogo