Drupal 9 readiness of Nodeaccess

Work updating the Grants form in terms of compliant SQL queries has started. After hammering, a lot of work on the grants of roles seems to be working but individual permissions still needs work. Most errors have been eliminated on the GrantsForm, there is much work to be done. Current work pushed to the dev branch.

Are the Drupal 9 compatibility issues still being worked on?

This is the only module keeping me from upgrading to Drupal 9.

Thank you.

Hello Team,
We are using nodeaccess in one of the sites in drupal which is to be ported to drupal 9.
Can anyone please release a stable version.
Thanks and Regards,
Sagar Devde

Me too after a Drupal 9 stable version upgrade.

Any news on a D9 stable release?

Hello alisonjo315,

I would like to be co-maintainer of this module.
Would provide me co-maintainership of this module.

Thanks and Regards,
Sagar Devde

Any Update regarding availablity of Node Access in Drupal 9?

This module provides critical functionality for a site I maintain. As we prepare for upgrading the site from Drupal 8 to 9, we look forward to any updates on a D9-ready version of nodeaccess. Thank you.

Maybe considering Sagar Devde's offer to co-maintain the module would help resume progress.

I'm encountering the same concern as #16. We are also in the final preparations for upgrading our site to D9 and this module is crucial for our site as well.

I would appreciate an estimation when D9 stable release will be available.

+1 on getting a release for this

One of my Drupal 9 instances relies on this module heavily, I would like to be a co-maintainer of this module too.

Adding another one

One more

@alisonjo315, replied on slack, thanks!

Would someone please post the status for release of nodeaccess in Drupal 9? We need to plan and would appreciate this information. Thanks!

I am using the upgrade_status module for Drupal 9 readiness. After adding the patch, it was compatible for D9. Works for me!

Made two commits, one for #3070372: Remove deprecate code, the other for #3169639: D9: Individual node permissions don't work.

Commented to both #3185156: Permission to individual Users throws error on grants tab of nodes and #3061378: Table name typo? which are not blockers anymore, I think.

@alisonjo315, is it time to make an alpha release for Drupal 9?

Adding #3236465: Improve issue 3145629 -- coding standards further to Nice to have.

Add two committed to Nice to have.

I am using the upgrade_status module for Drupal 9. After adding the patch , I have 6 errors :

CONTRIBUTED PROJECTS
--------------------------------------------------------------------------------
Nodeaccess 8.x-1.0
Scanned on mar 21 sep 2021.

5 errors found. 1 warning found. Avoid some manual work by using drupal-rector
for fixing issues automatically or Upgrade Rector to generate patches.

Help port this project at #3130778: Drupal 9 readiness of Nodeaccess

modules/contrib/nodeaccess/src/Form/GrantsForm.php:
┌──────────┬──────┬──────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼──────────────────────────────────────────────────────────────┤
│ Fix now │ 44 │ Call to deprecated function db_query(). Deprecated in │
│ │ │ drupal:8.0.0 and is removed from drupal:9.0.0. Instead, get │
│ │ │ a database connection injected into your service from the │
│ │ │ container and call query() on it. For example, │
│ │ │ $injected_database->query($query, $args, $options); │
│ │ │ │
│ Fix now │ 69 │ Call to deprecated function db_query(). Deprecated in │
│ │ │ drupal:8.0.0 and is removed from drupal:9.0.0. Instead, get │
│ │ │ a database connection injected into your service from the │
│ │ │ container and call query() on it. For example, │
│ │ │ $injected_database->query($query, $args, $options); │
│ │ │ │
│ Fix now │ 99 │ Call to deprecated function db_query(). Deprecated in │
│ │ │ drupal:8.0.0 and is removed from drupal:9.0.0. Instead, get │
│ │ │ a database connection injected into your service from the │
│ │ │ container and call query() on it. For example, │
│ │ │ $injected_database->query($query, $args, $options); │
│ │ │ │
│ Fix now │ 339 │ Call to deprecated function db_insert(). Deprecated in │
│ │ │ drupal:8.0.0 and is removed from drupal:9.0.0. Instead, get │
│ │ │ a database connection injected into your service from the │
│ │ │ container and call insert() on it. For example, │
│ │ │ $injected_database->insert($table, $options); │
│ │ │ │
│ Fix with │ 351 │ Call to deprecated function drupal_set_message(). Deprecated │
│ rector │ │ in drupal:8.5.0 and is removed from drupal:9.0.0. Use │
│ │ │ Drupal\Core\Messenger\MessengerInterface::addMessage() │
│ │ │ instead. │
│ │ │ │
└──────────┴──────┴──────────────────────────────────────────────────────────────┘

modules/contrib/nodeaccess/nodeaccess.info.yml:
┌──────────┬──────┬──────────────────────────────────────────────────────────────┐
│ STATUS │ LINE │ MESSAGE │
├──────────┼──────┼──────────────────────────────────────────────────────────────┤
│ Vérifie │ 0 │ Add core_version_requirement: ^8 || ^9 to designate that the │
│ r │ │ module is compatible with Drupal 9. See │
│ manuelle │ │ https://drupal.org/node/3070687. │
│ ment │ │ │

Does it exist another patch ?

Re #32, I think you were not using the dev version of the module, 8.x-1.0 used instead.

See https://www.drupal.org/project/nodeaccess/releases/8.x-1.x-dev, or run composer require 'drupal/nodeaccess:1.x-dev@dev' before applying the patch.

Thanks Jungle !I wasn't using the dev version.!It works.

While the DI is a nice to have, is it a dependency for a solid release for 9.x? It seems like we would be able to cut a beta or full release with the previous changes without the need for the DI Changes?

Or if they are a hard requirement, should we move them under that?

@jungle Can we please get a 9.x alpha release? I'm would prefer not to switch to a -dev release. There's just too many horror stories with those! 😂

Sorry, @dasginganinja, I only have the commit permission. But I had left @alisonjo315 a message to do it around a month ago. She replied me that she did put it into her to-do list.