Skip to content
This repository has been archived by the owner on Mar 7, 2018. It is now read-only.

Add CMIS storage support. #4

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

Conversation

fmerges
Copy link

@fmerges fmerges commented May 11, 2014

The following changes add support for CMIS.

There is a particular case which doesn't work with the current version of cmislib 0.5.1 unless patched, see: https://issues.apache.org/jira/browse/CMIS-799

The case consists of writing to a file with subfolders that doesn't exists in the remote repository. Example:

filename = '/folder/subfolder/some_text.txt'    # new file doesn't exist yet remotely
with storage.open(filename, 'w') as f:
    f.write(u'Hello World!\n')

In order to make it work you just have to apply the following diff to cmislib.model.py:

2912c2912
< return ResultSet(self._cmisClient, self, result)> return ResultSet(self._cmisClient, self._repository, result)

@jpvanhal
Copy link
Owner

Thanks! I'll try to review this over the weekend.

@mehcode
Copy link

mehcode commented May 15, 2014

For whats it worth we have a python 3.x only version of cmislib here: https://github.com/concordusapps/python-cmislib

We migrated it in a hurry a good while back.

@fmerges
Copy link
Author

fmerges commented May 15, 2014

Your link is a fork of jpots unreleased version https://github.com/apache/chemistry-cmislib right?

A made this taking the released one which is 0.5.1

I wonder what happen with it because I don't see much activity on the official page, I filed a bug report but nothing happened so far... to me it's curious because there are quite some companies that rely on Alfresco and others...

@Fiveside
Copy link

The python3 fork was made 6 months ago with whatever was on the subversion trunk at the time (the converter left some weird branches in the git version that I haven't bothered to clean up yet). As far as I can tell, no new changes have been pushed to cmislib in the time since then.

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

Successfully merging this pull request may close these issues.

4 participants