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

Use oAuth with getExpose() #42

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

Conversation

jcsiegrist
Copy link

getExpose() did not work with Objects published only for "Hompage" at IS24. With oAuth getExpose can retrieve such Objects and also "IS24" published Objects. Downside is the App needs to be certified.

I also disabled session_start(), since this results in a E_NOTICE since php 4.3 if a Session is already started. The App code should start the session, should it not?

The OAuth class just defined the OAuthException class without checking if it already exists. This resulted in a fatal error if the PHP oAth extension is installed.

Jean-Claude Siegrist added 3 commits December 13, 2016 11:20
to avoid E_NOTICE as of php 4.3 when session is already started
so Objects published only for „Hompage“ on IS24 can be retrieved. Should work with IS24 published Objects as well. App has to be zertified
before trying to redeclare it. Otherwise fatal php error is thrown if php oAuth extension is installed.
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.

1 participant