Problem/Motivation

The logic for photos_access gets mixed up when switching between options. Sometimes multiple entries will be in {photos_access_album} for the same nid and this will cause problems with the views filter. Also the password is wiped out in various places. One if the password field is left blank when updating the node after a password has been set.

Proposed resolution

Keep one row per node in {photos_access_album} and update as needed.

Remaining tasks

  • Test switching between all options while keeping an eye on the {photos_access_album} table and {node_access}.
  • Check if password is empty before updating. Make update query dynamic.

Comments

Nathaniel created an issue. See original summary.

Nathaniel’s picture

This might require some restructuring.

Remove {photos_access_album}.updateid and add {photos_access_user}.collaborate.

Nathaniel’s picture

Backporting to 7.x-3.x.

Need to update 8.x-4.x migration files.

  • Nathaniel committed dfa7a33 on 7.x-3.x
    Issue #2734461 by Nathaniel: Fix photos_access to only store one record...

  • Nathaniel committed 6acc133 on 8.x-4.x
    Issue #2734461 by Nathaniel: Fix photos_access to only store one record...

  • Nathaniel committed f86c9a4 on 8.x-4.x
    Issue #2734461 by Nathaniel: Update migration files to work with new db...
Nathaniel’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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