Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Inside cmis_sync/cmis_sync.cmis.inc, the _cmis_sync_cmis_drupal_handle_updates uses the variable $repository, but in the function, $cmis_repository is used.
$cmis_repository should be changed to $repository - this occurs twice
* Creates/updates Drupal nodes with CMIS content.
*
* @param $repository
* @param $sync_map_type
* @param $node_type
*/
function _cmis_sync_cmis_drupal_handle_updates($repository, $sync_map_type, $node_type) {
// get CMIS object properties
if (isset($sync_map_type['cmis_folderId'])) {
$cmis_folder = cmisapi_getProperties($cmis_repository->repositoryId, $sync_map_type['cmis_folderId']);
}
elseif (isset($sync_map_type['cmis_folderPath'])) {
$cmis_folder = cmisapi_getObjectByPath($cmis_repository->repositoryId, $sync_map_type['cmis_folderPath']);
}
else {
throw new CMISException(t("Please set `cmis_folderPath` or `cmis_folderId` properties for [@type] Drupal type.", array(
'@type' => $node_type
)));
}
Comment | File | Size | Author |
---|---|---|---|
#3 | cmis.1169698.3.patch | 1.03 KB | tobiasb |
#2 | cmis.1169698.3.patch | 1.03 KB | tobiasb |
Comments
Comment #1
soundasleep CreditAttribution: soundasleep commentedConfirmed in separate install -- a quick workaround is simply to add the following at the start of that function:
$cmis_repository = $repository;
Otherwise, the following error occurs:
Comment #2
tobiasbComment #3
tobiasb* Fixed a typo
Comment #4
IanNorton CreditAttribution: IanNorton commentedComment #5
IanNorton CreditAttribution: IanNorton commentedFixed in latest commit - thanks to tobiasb for the patch