Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement getAcl method #75

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sascha-egerer
Copy link
Contributor

No description provided.

@sascha-egerer sascha-egerer mentioned this pull request Aug 18, 2020
@sascha-egerer sascha-egerer force-pushed the feature-get-acl branch 2 times, most recently from 262932c to a6fc78e Compare August 18, 2020 06:20
@wallon-ines
Copy link

You forgot AbstractCmisObject modification in your pr. but with this modification It's work

diff --git a/src/DataObjects/AbstractCmisObject.php b/src/DataObjects/AbstractCmisObject.php
index 727807e..9477777 100644
--- a/src/DataObjects/AbstractCmisObject.php
+++ b/src/DataObjects/AbstractCmisObject.php
@@ -747,7 +747,8 @@ abstract class AbstractCmisObject implements CmisObjectInterface
      */
     public function getAcl()
     {
-        return $this->acl;
+        $result = $this->getSession()->getAcl($this, false);
+        return $result;

     }
 
     /**

In my Drupal, it's work too :

Capture d’écran_2020-08-18_09-22-11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants