Problem/Motivation

I'm installing the module as part of an installation profile of a distribution. Because it is now always flagging the node access rebuild in view_unpublished_install(), the very first thing the people see after installing my distribution is the false error message that they need to rebuild access permissions. This is not even true since there is no content available in the site at this point.

Proposed resolution

Check if there is actually unpublished content available in the database before flagging the error.

Remaining tasks

Implement the fix.

User interface changes

One less error message for the administrators to worry about.

API changes

None.

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen created an issue. See original summary.

amaria’s picture

Feel free to create a patch for this.

DuaelFr’s picture

Status: Active » Needs review
FileSize
914 bytes

There you go :)
Note: a better name for the helper function is welcome

claudiu.cristea’s picture

Status: Needs review » Reviewed & tested by the community

  • amaria committed a4760ac on 8.x-1.x authored by DuaelFr
    Issue #2837685 by DuaelFr, pfrenssen: Do not unnecessarily flag a...
amaria’s picture

Status: Reviewed & tested by the community » Fixed

Thanks. I committed this, so marking as fixed. Although, I still got the message to rebuild when I had no unpublished nodes. But I was installing/uninstalling a lot so it could just be me :) However, if you're still having issues, feel free to re-open with details.

Status: Fixed » Closed (fixed)

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