Attached is the D7 and D9 exports of one of the views on my site, "Content advanced" which provides a more robust interface for content moderators to find content.

The original view required the "Administer content" permission to access the View.

The converted view only requires "View published content"

This grants anonymous users access to the contents of the view, which may contain private/unpublished content. No thank you. :)

Comments

webchick created an issue. See original summary.

sundhar’s picture

Hi webchick,

getPluginList method changed in viewsMigration plugin. Views::PluginList funtion not gives all defined plugins, example role access plugin, Its defined in user module. So, I change the rendering method to get view plugin list.

This issue related to https://www.drupal.org/project/views_migration/issues/3210462
It had solved with viewsFilterPlugins-3210462-7.patch (https://www.drupal.org/files/issues/2021-05-02/viewsFilterPlugins-3210462-7.patch)

sundhar’s picture

Status: Active » Needs review
StatusFileSize
new625 bytes

Assign 'administer nodes' permission if permission is null.

  • Jeya sundhar committed 43d828e on 1.1.x
    Issue #3211486 by Jeya sundhar, webchick: Views permissions not...
webchick’s picture

Hmmm. I wonder if there's another indicator we could look for here. For example, if permissions are null, AND there's no ->published = TRUE filter on the View? That seems like it would be more accurate to the intent.

It is strange that it's blanking out the perm altogether. Hm.

sundhar’s picture

StatusFileSize
new648 bytes

Hmm.. yes,
We change the permission to 'access content'.
And In D7 views module default access permission is 'access content'. programmatic created views has the access permission is null. so we assign the 'access content' permission. If the access permission is null.

  • Jeya sundhar committed 854adb9 on 1.1.x
    Issue #3211486 by Jeya sundhar, webchick: Views permissions not...
webchick’s picture

webchick’s picture

Status: Needs review » Fixed
StatusFileSize
new135.57 KB

YES! :D Problem solved!

Status: Fixed » Closed (fixed)

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