This access rule plugin checks entity existence (this means entity ID existence).

CommentFileSizeAuthor
#4 ctools-entity_exists-2183587-4.patch2.19 KBmaximpodorov
#1 ctools-entity_exists-2183587.patch2.27 KBmaximpodorov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

maximpodorov’s picture

maximpodorov CreditAttribution: maximpodorov commented
FileSize
ctools-entity_exists-2183587.patch2.27 KB
maximpodorov’s picture

maximpodorov CreditAttribution: maximpodorov commented

This is the same as in
https://drupal.org/project/ctools_entity_exists
module.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented 
  1. +++ b/plugins/access/entity_exists.inc
@@ -0,0 +1,60 @@
+    $child_plugin['title'] = $entity['label'] .  ': ' . t('exists');

    Suppose better check_plain($entity[label]) 

  2. +++ b/plugins/access/entity_exists.inc
@@ -0,0 +1,60 @@
+  return (!is_null($entity_id));
...
+  $entity_id = isset($entity->{$info['entity keys']['id']}) ? $entity->{$info['entity keys']['id']} : NULL;

    !empty($entity->{$info['entity keys']['id']}) should be enough

maximpodorov’s picture

maximpodorov CreditAttribution: maximpodorov commented
FileSize
ctools-entity_exists-2183587-4.patch2.19 KB

Your fixes are applied.

DamienMcKenna queued 4: ctools-entity_exists-2183587-4.patch for re-testing.

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented

The 5 year old patch in #4 to entity_exists.inc still applies cleanly to the latest ctools 7.x-1.x-dev, but needs to be officially reviewed and tested by the community.

Applied patch plugins/access/entity_exists.inc cleanly.