DynamicFKAuthorization - Add comment

diff --git a/Civi/API/Subscriber/DynamicFKAuthorization.php b/Civi/API/Subscriber/DynamicFKAuthorization.php
index 4aeef4c314cd40b88525b394b59dd900e5be4520..3b7b83b2395ad8577093dd7795e3840f2870a2cd 100644 (file)
--- a/Civi/API/Subscriber/DynamicFKAuthorization.php
+++ b/Civi/API/Subscriber/DynamicFKAuthorization.php
@@ -73,7 +73,7 @@ class DynamicFKAuthorization implements EventSubscriberInterface {
   protected $actions;
 
   /**
-   * @var string, SQL; a query which looks up the related entity
+   * @var string, SQL. Given a file ID, determine the entity+table it's attached to.
    *
    * ex: "SELECT if(cf.id,1,0) as is_valid, cef.entity_table, cef.entity_id
    * FROM civicrm_file cf

diff --git a/Civi/Core/Container.php b/Civi/Core/Container.php
index 7de921c8891daf8be49482f16397728543da7540..182737e712f4eb6f5ba51862ab8b138397c08499 100644 (file)
--- a/Civi/Core/Container.php
+++ b/Civi/Core/Container.php
@@ -162,6 +162,7 @@ class Container {
       $kernel,
       'Attachment',
       array('create', 'get', 'delete'),
+      // Given a file ID, determine the entity+table it's attached to.
       'SELECT if(cf.id,1,0) as is_valid, cef.entity_table, cef.entity_id
          FROM civicrm_file cf
          LEFT JOIN civicrm_entity_file cef ON cf.id = cef.file_id

diff --git a/api/v3/Attachment.php b/api/v3/Attachment.php
index 258c4525d22766dafe1c73343297674f89083285..5bc789282b5913b44b08b67e6217b532528a1587 100644 (file)
--- a/api/v3/Attachment.php
+++ b/api/v3/Attachment.php
@@ -103,6 +103,7 @@ function _civicrm_api3_attachment_create_spec(&$spec) {
  * @return array
  *   Array of newly created file property values.
  * @throws API_Exception validation errors
+ * @see Civi\API\Subscriber\DynamicFKAuthorization
  */
 function civicrm_api3_attachment_create($params) {
 

diff --git a/tests/phpunit/Civi/API/Subscriber/DynamicFKAuthorizationTest.php b/tests/phpunit/Civi/API/Subscriber/DynamicFKAuthorizationTest.php
index 355193688c89d7d5a9cb5aceca86ed44d9ab9ffb..c6ca9ea4e252d461bd5ebfd52b71f34a6224ae79 100644 (file)
--- a/tests/phpunit/Civi/API/Subscriber/DynamicFKAuthorizationTest.php
+++ b/tests/phpunit/Civi/API/Subscriber/DynamicFKAuthorizationTest.php
@@ -77,6 +77,7 @@ class DynamicFKAuthorizationTest extends \CiviUnitTestCase {
       $this->kernel,
       'FakeFile',
       array('create', 'get'),
+      // Given a file ID, determine the entity+table it's attached to.
       "select
       case %1
         when " . self::FILE_WIDGET_ID . " then 1