Closed (outdated)
Project:
Content locking (anti-concurrent editing)
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
10 Dec 2015 at 16:12 UTC
Updated:
20 Oct 2025 at 09:53 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
amcgowanca commentedComment #3
alberto56 commentedin your patch, content_lock_inline_entity_form_form_alter() calls _content_lock_is_lockable_node() with a node which is not guaranteed to have a nid, which can cause an undefined unless you have added the patch #2682781: In content_lock_content_lock_node_lockable(), do not assume the node has a nid (PHP Undefined property: stdClass::$nid in content_lock.module on line 984).
Comment #4
alberto56 commentedNice patch, here is a new version with a few changes:
(1) the patch adds ./content_lock/modules/content_lock_inline_entity_form to ./content_lock, so we end up with ./content_lock/content_lock/modules/content_lock_inline_entity_form if we apply the patch
(2) I think version information is added by the Drupal packaging system, so it should not be added to new modules in a project.
(3) As stated in the previous comment, until #2682781: In content_lock_content_lock_node_lockable(), do not assume the node has a nid (PHP Undefined property: stdClass::$nid in content_lock.module on line 984) makes it in, we need to avoid calling _content_lock_is_lockable_node() with a node which does not have a nid.
Comment #5
pandaski commentedComment #6
pandaski commentedHappy to see it in next release
Comment #7
damienmckennaComment #8
damienmckennaPlease try to remember to set the issue status to "needs review" when you upload a patch. Thanks.
Comment #9
astonvictor commentedD7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.