TypeError in RevokeAccess::updateSettings

boromino → closed merge request !1

That fixes the issue, but maybe it would be better to use content_access_get_per_node_settings() and content_access_save_per_node_settings() respectively.

Thanks @boromino for this report. Yes, I wanted to use that other method originally as well but ran into issues that I couldn't resolve.

Your MR looks good as a starting point, but I think there are a couple more things that also need to be done:

• There are 2 more instances of unserialize, I guess they should be changed too?
• In the 1.x branch we probably also need to define that version 2 of content_access is required, but that's only needed for the backport then

There are 2 more instances of unserialize, I guess they should be changed too?

No, because there config is loaded which is actually serialized.

In the 1.x branch we probably also need to define that version 2 of content_access is required, but that's only needed for the backport then

Version 2 of content access is already required in 1.x branch (my task description may be confusing in so far).

OK, got it. I've merged this into 2.0.x and will now backport to 1.0.x and publish a new release.

Version 2 of content access is already required in 1.x branch

Yes, in the composer.json but not in the info file. I've added it there now too and published release 1.0.2

Automatically closed - issue fixed for 2 weeks with no activity.