entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

misc: drop anonymous attribute from users (#77162) #279

Merged
fpeters merged 1 commits from wip/77162-user-drop-anonymous-column into main 2023-05-02 08:16:59 +02:00
Conversation 0 Commits 1 Files Changed 7 +11 -34
fpeters commented 2023-05-01 10:53:18 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/77162-user-drop-anonymous-column from 8f3e6983c0 to 2a386466ed 2023-05-01 11:14:44 +02:00 Compare
fpeters force-pushed wip/77162-user-drop-anonymous-column from 2a386466ed to 158ed09837 2023-05-01 11:38:37 +02:00 Compare
fpeters force-pushed wip/77162-user-drop-anonymous-column from 158ed09837 to 85655b57b1 2023-05-01 12:05:23 +02:00 Compare
fpeters force-pushed wip/77162-user-drop-anonymous-column from 85655b57b1 to b2030b0f20 2023-05-01 13:06:54 +02:00 Compare
fpeters reviewed 2023-05-01 13:32:09 +02:00
wcs/api.py
@ -303,3 +304,2 @@
is_admin = True
anonymous = True
is_api_user = False
is_api_user = True
fpeters commented 2023-05-01 13:29:58 +02:00
Author
Owner
Copy Link

Ça supprime le anonymous ici, qui était le seul endroit où c'était True, ça remet aussi is_api_user à True, ça avait été changé dans #53926 mais a posteriori c'était une mauvaise idée et il suffit d'une petite modif à wcs/qommon/http_request.py pour passer.

Ça supprime le anonymous ici, qui était le seul endroit où c'était True, ça remet aussi is_api_user à True, ça avait été changé dans #53926 mais a posteriori c'était une mauvaise idée et il suffit d'une petite modif à wcs/qommon/http_request.py pour passer.
wcs/forms/common.py
@ -202,2 +196,2 @@
if session and session.is_anonymous_submitter(self.filled):
mine = True
mine = self.filled.is_submitter(user)
fpeters commented 2023-05-01 13:30:44 +02:00
Author
Owner
Copy Link

Le is_submitter(...) fait déjà le boulot de regarder dans la session, inutile de répéer ça ici.

Le is_submitter(...) fait déjà le boulot de regarder dans la session, inutile de répéer ça ici.
wcs/forms/common.py
@ -370,3 +361,1 @@
if str(session.user) != str(filled.user_id):
raise errors.AccessUnauthorizedError()
else:
if not self.filled.is_submitter(get_request().user):
fpeters commented 2023-05-01 13:31:06 +02:00
Author
Owner
Copy Link

(bis) Le is_submitter(...) fait déjà le boulot de regarder dans la session, inutile de répéer ça ici.

(bis) Le is_submitter(...) fait déjà le boulot de regarder dans la session, inutile de répéer ça ici.
wcs/sql.py
@ -3231,3 +3226,3 @@
o.deleted_timestamp,
o.is_active,
) = (str_encode(x) for x in tuple(row[:13]))
) = (str_encode(x) for x in tuple(row[: len(cls._table_static_fields)]))
fpeters commented 2023-05-01 13:31:51 +02:00
Author
Owner
Copy Link

Il y a déjà assez d'endroits à modifier pour supprimer un attribut, en voici un de moins.

Il y a déjà assez d'endroits à modifier pour supprimer un attribut, en voici un de moins.
fpeters changed title from WIP: misc: drop anonymous attribute from users (#77162) to misc: drop anonymous attribute from users (#77162) 2023-05-01 13:32:19 +02:00
lguerin approved these changes 2023-05-01 19:21:23 +02:00
fpeters merged commit 45bee7910b into main 2023-05-02 08:16:59 +02:00
fpeters deleted branch wip/77162-user-drop-anonymous-column 2023-05-02 08:16:59 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#279
No description provided.
Delete Branch "wip/77162-user-drop-anonymous-column"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?