I just wanted to ask if there are any plans to migrate the module to Drupal 8? That would be nice :-)

Problem/Motivation

TODO

Proposed resolution

TODO

Remaining tasks

  • Stable D7 release & tests #1
  • Test driven development for D8 #1

#20:

For a port we should think about to:

  • Join efforts with protected_pages and eventually just add a UI based on that, because other functionality and configuration might be shared!
  • Implement this for (all kind of) entities and not only nodes. Since Drupal 8 we should think in entities and not in "node" vs. "user" vs. "taxonomy" vs. "eck" etc...

User interface changes

TODO

API changes

TODO

Data model changes

TODO

Release notes snippet

TODO

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grimreaper’s picture

Title: Drupal 8 » [Meta] Drupal 8 port
Category: Feature request » Task

Hello,

Thank you for your request, that is motivating. With Izus we started some IRL discussions, we want to be more generic and not be only focused on nodes, so there would be more users for the module and more developers interested in.

Some kind of protected API :)

But currently I am focused on helping porting search_api_attachments with him, it is almost done. I am also focused on some migrations in D8 and other stuffs.

On protected node, Before porting to D8, it would be cool to have a stable version for D7 :). There is a ******* automated test that does not want to be green with the testbot #2447913: Fix fork feature and there are still some other tests to implements #2277045: Implement tests for protected node.

So if you want to help us, you are welcome. :)

izus’s picture

Category: Task » Plan
izus’s picture

Status: Active » Postponed

i guess this should be postponed until we have a stable d7 release ;) #2105245: [META] Plan for Protected Node v7.x-1.0 release

digitgopher’s picture

I haven't dug into D8 yet, but +1 for some sort of protected API. I wonder if Protected Pages would be interested in this as well, if it would be possible.

Grimreaper’s picture

Status: Postponed » Active

D7 stable release is out. Back to active status.

bjlewis2’s picture

Yay! I was just looking for some way to password protect nodes. Hope there's something to test soon! :)

Anonymous’s picture

I'd like to try to help on this. What's the current status for the port? Will there be an 8.x dev branch soon?

Grimreaper’s picture

Hello,

Thank you all for your interest in the D8 version. The current state is that we need to discuss the structure of module for D8.

Maybe looking above the node, at the entity level, but we need to poke the feasibility.

Unfortunately we (Izus and I) had only some 5 minutes discussions IRL for the moment.

DamienMcKenna’s picture

Title: [Meta] Drupal 8 port » Port Protected Node to Drupal 8

Standardized the title.

hottomali’s picture

Just checking to see if anyone's migrated a version of this to 8.x yet? I'm quoting a DR6 migration and we'd prefer to go to 8, but there are a lot of dependencies on this module.

DamienMcKenna’s picture

@hottomali: I suspect you might need to include time to port some of this module's functionality as part of your upgrade estimate.

Vlad_Bo’s picture

Hi guys.

If you don't mind I will start porting module to D8.

apsylone’s picture

@Vlad_Bo Haved you begin the port to D8 ? Can we try to help you ?

alex.87’s picture

Whats the status of this, is there any progress?

EdPhillis’s picture

Looking forward to a Drupal 8 port...any news?

piridium’s picture

Looking forward to D8 port, too. :-)
Any status updates?

John_B’s picture

Does anyone have any code to share? I might have to do this for a client as there seems to be no real alternative to this module for Drupal 8. Any start on the code could save some time.

Chris Matthews’s picture

There were some commits to the 7.x-1.x branch today so maybe development on the 8.x branch will start soon.

Grimreaper’s picture

Hello,

To give some info, I personally don't have time to port this module on D8/D9 on my free time. And more over doing it alone.

I saw that some people said that they started some ports. I don't see any links in this issue, so I guess code is not available publicly or they changed their mind. No offense here.

In this comment, I speak (write precisely :)) for myself, I don't know the opinion of @izus about this.

So my opinion is if someone has a sandbox on drupal.org or Github to show, and if people test the sandbox and everything is ok, I am ok to open a dedicated branch and to put someone as an additional maintainer for the D8/D9 version. But this is not a decision I will take alone.

Also, if no one had done any sandboxes yet, and if people wants to sponsor this port, you can contact me. I am not a freelance, so for administrative and contract stuff, It will need to be done with my current employer.

Anybody’s picture

Issue summary: View changes

Hi all and just my two cents... we really loved protected_node!

For a port we should think about to:

  • Join efforts with protected_pages and eventually just add a UI based on that, because other functionality and configuration might be shared!
  • Implement this for (all kind of) entities and not only nodes. Since Drupal 8 we should think in entities and not in "node" vs. "user" vs. "taxonomy" vs. "eck" etc...

Thank you!

Anybody’s picture

Issue summary: View changes
Anybody’s picture

Issue summary: View changes

Updated issue summary with templates and put some information from the comments there.

izus’s picture

i agree completly with @Grimreaper's opinion on that :)

Grimreaper’s picture

Thanks @Anybody and @izus for your replies.

@Anybody: Yes, that's why since comment 1, I wrote about some kind of "Protected API". :)

meyerrob’s picture

Good to hear that this issue is still alive :-)

I could try to raise some money, but as we are only a very small project at an university that definitely won´t be enough to sponsor the port alone. But maybe there are some more of the 3K users of this module who could organize some funding?

zwerg’s picture

+1 for this! I think the benefit of this module is to decide on node creation whether it should be protected or not. Protected pages is less user-friendly ;-)

intrafusion’s picture

Seeing as there hasn't been any movement on this I thought I'd run the module (not protected_node_rules & protected_node_views) though Drupal 7 to 8/9 Module Upgrader and it created a log file and a massive patch (attached).

I hoped that this could be used as the basis for starting the port, perhaps just the new 8/9 branch and then any further work could be carried out in separate issues making it much easier to track.

intrafusion’s picture

Status: Active » Needs review
Anybody’s picture

Is there any active maintainer for this module? I agree that after a basic check of #27 it would make a lot of sense to create a new branch for Drupal 8 / 9 !

Grimreaper’s picture

Version: 7.x-1.x-dev » 1.0.x-dev

Hello,

Branch 1.0.x created.

You can work with fork and MR in issues.

Thanks.

Have fun.

Anybody’s picture

Status: Needs review » Fixed

Thank you very much @Grimreaper, so I think we should test the D8/9 version now and create follow-up issues to improve that. I guess we can close this issue.

@all: Would be cool if you could participate in that testing / improvement process :)

Status: Fixed » Closed (fixed)

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

Grimreaper’s picture

Hello,

For information, the port to Drupal 9 is happening in https://www.drupal.org/project/entity_access_password

Grimreaper’s picture

The alpha blockers had been fixed. So here is the first alpha version! https://www.drupal.org/project/entity_access_password/releases/1.0.0-alpha1

Happy testing!

Note: I will stop posting here updates on Entity Access Password as now a first tag had been made.

DamienMcKenna’s picture

That's awesome! Thank you!

Anybody’s picture

Whao thank you very very much @Grimreaper! Great work!

Grimreaper’s picture

Thanks!

But do not forget to thank @meyerrob and Ludwig-Maximilians-Universität München for sponsoring the port. :)

If people can give a look at #3260323: Use user password functions for easier migration from D7 to D8/9, this will greatly help data migration from D7.

There are also some documentation pages https://www.drupal.org/docs/contributed-modules/entity-access-password, do not hesitate to give it a look and to complete if needed.

meyerrob’s picture

Yes, thank you @Grimreaper for this fantasic port! And we should not forget that SMILE made our sponsorship possible with a offer that fitted our small budget :-)

Maybe the community could help us to finish the port by testing - there is still some time left in the next two weeks where @Grimreaper has some dedicated time for this project (which is planned to be used for writing a D7-D9 migration and automatic tests at the moment)