Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shkiper’s picture

I've made a Drupal 8 version of this module. Could someone please review my patch?

fornigam’s picture

Hi Shkiper,

Could you please provide sandbox or other link for download 'Node Edit Protection' in Drupal 8 ?

floretan’s picture

Version: 7.x-1.0 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community
FileSize
19.38 KB

The patch from @shkiper in comment #1 works (tested with simple text fields and with CKeditor). I also had a look at the code, everything looks good to me.

There was a warning about trailing whitespace when applying the patch, here's a functionally identical patch with the line endings fixed so that no warnings are thrown.

  • czigor committed 902339c on 8.x-1.x authored by shkiper
    Issue #2073503 by thehong, shkiper, floretan: Upgrade to Drupal 8
    
czigor’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for all the work on this! I have created a 8.x-1.x branch and a 8.x-1.x-dev release. Hopefully it will appear on the module page in the next 12 hours.

Status: Fixed » Closed (fixed)

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