#6982 (Automatically set the owner of new uploads)

by Luc Saffre · in Developers · assigned to Luc Saffre

When a user uploads a file from the dashboard, Lino Patrols should attached the upload automatically to the calendar event the user is working on.

1 mois, 4 semaines ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) : I now pushed the test results. There was even a typo bug in my code this morning.
2 mois ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) : I pushed my changes for this already before running the test suites, that's a bit too quickly... because today I will move to Tallinn...
2 mois ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) : I currently have both implementations, they both work, now I must decide which one to choose...
2 mois ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) : Side effect: the change on_ui_created signal handler now expects ar instead of request.
2 mois ago by Luc Saffre in reply to Luc Saffre about #6982 (Automatically set the owner of new uploads) : But what about the on_ui_created signal?
2 mois ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) (1 reply) : A first implementation uses a new plugin method get_owner_for_new_upload() which returns None, and the patrols plugin replaces this by its own callable. That's a bit hackerish. I'm still thinking about a more straightforward approach. But a Django signal would get called too late, when potential not...
2 mois ago by Luc Saffre about #6982 (Automatically set the owner of new uploads) : Created new Ticket #6982 (Add a customizable hook for setting Upload.owner).