Internal Server Error 500 after two consecutive PATCH requests

[gcakir]

While adding data to the draft, the first PATCH request successfully adds data to the existing draft. But if the same request is made with the identical payload/data (e.g. joe@abc.def) for the same path (e.g. contact_email), Internal Server Error (500) will be returned from the API.

data_patch = [

    {"op": "add", "path": "/contact_email", "value": "joe@abc.def"},

]

This seems to be happening because publication state disappears in

b2share/b2share/modules/deposit/permissions.py

Line 323 in 8834415

del new_deposit['publication_state']

Could it be that whole block in this part is indented one level less than it should be?

@JohannesLares
@hjhsalo

[hjhsalo]

The indent theory has been briefly tested and it seems that fixing indentation fixes the problem.
Full unit test suite was not yet run. @gcakir do you want to open a pull request?

[gcakir]

The indent theory has been briefly tested and it seems that fixing indentation fixes the problem. Full unit test suite was not yet run. @gcakir do you want to open a pull request?

Pull request created. 👍