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

Update SugarCase.php #32

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

Update SugarCase.php #32

wants to merge 1 commit into from

Conversation

pgorod
Copy link

@pgorod pgorod commented Nov 30, 2018

@DWilly92
Copy link

DWilly92 commented Aug 4, 2020

Why has this pull request not been accepted yet? It has been over 2 years and it's still an issue to this day. This solves the issue 100%, and without it the Joomla component does not work... I'm tempted to fork this project and create my own that works properly if this one is as dead as it seems.

@pamchi
Copy link

pamchi commented Mar 22, 2022

After done this, i was unable to see de details of a case, finally i did this: https://community.suitecrm.com/t/simple-fix-for-advanced-openportal-with-php-7-1/46966 and works well

@pgorod
Copy link
Author

pgorod commented Mar 22, 2022

This patch here is supposed to be the same change proposed in that Forum post you linked. What exactly did you do, additional to the change in this patch?

@chris001
Copy link

Simple one line fix (set a function parameter's default type) without which the portal fails! This one should obviously be reviewed and if accepted, merge immediately and auto build a hotfix release version.

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.

4 participants