Fix issue with document creation caused by changes in getTypeDefinition

Story ID:2295472. Update KT CMIS implementation to 1.0 lib/api/compliance Committed by: Paul Barrett

Fix issue with document creation caused by changes in getTypeDefinition
Story ID:2295472. Update KT CMIS implementation to 1.0 lib/api/compliance Committed by: Paul Barrett
Paul Barrett
1 parent bb935276
Showing 4 changed files with 5 additions and 14 deletions
lib/api/ktcmis/services/CMISObjectService.inc.php
lib/api/ktcmis/services/CMISRepositoryService.inc.php
webservice/atompub/cmis/KT_cmis_atom_server.services.inc.php
webservice/classes/atompub/cmis/KT_cmis_atom_server.inc.php
@@ -70,8 +70,7 @@ class CMISObjectService {
             throw new ConstraintViolationException('Object base type could not be determined. ' . $e->getMessage());
         }
  
-        if ($typeDefinition['attributes']['baseId'] != 'cmis:document')
-        {
+        if ($typeDefinition['attributes']['baseId'] != 'cmis:document') {
             throw new ConstraintViolationException('Object is not of base type document');
         }
  
@@ -306,8 +305,6 @@ class CMISObjectService {
     //      an ACE provided which is not supported by the repository.
     public function createFolder($repositoryId, $properties, $folderId, $policies = array(), $addACEs = array(), $removeACEs = array())
     {
-        global $default;
-        $default->log->info('try create folder');
         $objectId = null;
  
         // fetch type definition of supplied type and check for base type "folder", if not true throw exception
@@ -347,7 +344,7 @@ class CMISObjectService {
         if (!$typeDefinition['attributes']['controllableACL'] && (count($addACEs) || count($removeACEs))) {
             throw new ConstraintViolationException('This object-type does not support ACLs');
         }
-        $default->log->info(print_r($properties, true));
+        
         // set title and name identical if only one submitted
         if ($properties['title'] == '') {
             $properties['title'] = $properties['name'];
@@ -567,7 +564,7 @@ class CMISObjectService {
         if (!$exists) {
             throw new updateConflictException('Unable to delete the object as it cannot be found.');
         }
-        global $default;
+        
         // throw ConstraintViolationException if method is invoked on a Folder object that contains one or more objects
         if ($typeId == 'cmis:folder')
         {
@@ -163,9 +163,6 @@ class CMISRepositoryService {
     // NOTE this code may fit better in the Repository Class
     function getTypeDefinition($repositoryId, $typeId)
     {
-        global $default;
-        $default->log->debug();
-        $default->log->info(str_replace('cmis:', '', $typeId));
         $typeId = ucwords(str_replace('cmis:', '', $typeId));
         $object = 'CMIS' . $typeId . 'Object';
  
@@ -183,7 +180,7 @@ class CMISRepositoryService {
         // NOTE The specification is ambigous here: it states that this function must return the type properties, but 
         //      the atompub example shows the type attributes, not properties;  since most properties are only populated
         //      on creation of an instance of an object-type, we choose to go with the attributes and not the properties
-        $typeDefinition = $cmisObject->getAttributes();
+        $typeDefinition['attributes'] = $cmisObject->getAttributes();
  
         return $typeDefinition;
     }
@@ -709,7 +709,7 @@ class KT_cmis_atom_service_type extends KT_cmis_atom_service {
             return null;
         }
  
-        $feed = KT_cmis_atom_service_helper::getTypeFeed($type, array($typeDefinition));
+        $feed = KT_cmis_atom_service_helper::getTypeFeed($type, array($typeDefinition['attributes']));
  
         // Expose the responseFeed
         $this->responseFeed=$feed;
@@ -166,9 +166,6 @@ class KT_cmis_atom_server extends KT_atom_server {
             header('Content-type: text/xml');
         }
  
-//        global $default;
-//        $default->log->info($this->output);
-        
         if ($this->renderBody) {
             echo $this->output;
         }