Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Make oczpush compatible with OC5 contacts app #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

icewind1991
Copy link

Makes oczpush work against OC git master,
will make it incompatible with 4.5 and older

@gza

@gza
Copy link
Owner

gza commented Feb 20, 2013

thanks for the patch, I'll test it when I will upgrade OC to 5.0
@todo : do the same for occalendar.php

@icewind1991
Copy link
Author

As far as I know the api for calendar hasn't changed, did a quick test for both calendar and contacts against an android client and calendar seemed to work fine without any changes.

I'll do some more testing one I got my new phone where I need this for.

gza notifications@github.com wrote:

thanks for the patch, I'll test it when I will upgrade OC to 5.0
@todo : do the same for occalendar.php


Reply to this email directly or view it on GitHub.

@bogaurd
Copy link

bogaurd commented Apr 21, 2013

It looks as though oczpush does not currently work with OC5. I am running 5.0.5 & see the following error when I try to access oczpush:

PHP Fatal error: Class 'OC_Contacts_Addressbook' not found in /var/www/oczpush/backend/occontacts.php on line 54

Edit:

oczpush is now running, after changing a few names:

sed -i 's/OC_Contacts_/OCA\Contacts/' backend/occontacts.php

@mazprivat
Copy link

@bogaurd
hi there, I was using the file provided by icewind1991 successfully with OC 5.0.4. Is Your change any different than the patch that was provided in first place, does OC 5.0.5 change folders again?
(successfully syncing Nokia Symbian Belle with OC 4.5 on Synology DS without this patch also)

@bogaurd
Copy link

bogaurd commented Apr 21, 2013

@mazprivat
I didn't realize icewind1991's commit hadn't been merged - I just did the same thing, so no, 5.0.5 does not change folders again. :-)

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