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

Issue with DetailView of Email module on 7.10.7 xampp #6131

Closed
jamiemurray opened this issue Jul 8, 2018 · 4 comments
Closed

Issue with DetailView of Email module on 7.10.7 xampp #6131

jamiemurray opened this issue Jul 8, 2018 · 4 comments
Assignees
Labels
Area: Emails Issues & PRs related to all things regarding emails & email module Priority:Critical Issues & PRs that are critical; broken core functionality, fatal errors - there are no workarounds Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type:Bug Bugs within the core SuiteCRM codebase

Comments

@jamiemurray
Copy link

DetailView of email module shows the raw php code on Xampp on windows

Issue

Since upgrading... detailview of an email is broken on xampp using windows (have not tested on linux)

Expected Behavior

When clicking into the detailview of an email... the email content should display

Actual Behavior

The raw php code is displayed.

Possible Fix

I fixed by reverting change made on modules/emails/vardefs.php with commit a60c60f and running quick repair

Steps to Reproduce

Upgrade on windows xampp to v 7.10.7 from previous version and go to detailview of any outbound or inbound email

Your Environment

Suite: 7.10.7 upgrade from 7.10.6
Windows 10 XAMPP PHP 7

@Dillon-Brown Dillon-Brown added Type:Bug Bugs within the core SuiteCRM codebase Priority:Critical Issues & PRs that are critical; broken core functionality, fatal errors - there are no workarounds Area: Emails Issues & PRs related to all things regarding emails & email module labels Jul 10, 2018
@countrypark
Copy link

countrypark commented Jul 19, 2018

Hi, the fix mentioned above resolved the broken page issue, however it brought back issue #4035 - "Upgrade on 7.9.4 Content in emails are same in body for all emails."

@Dillon-Brown Dillon-Brown added the Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution label Jul 22, 2018
@Dillon-Brown Dillon-Brown self-assigned this Jul 22, 2018
@samus-aran
Copy link
Contributor

Hi everyone! @countrypark and @jamiemurray We have proposed a fix in this Pull Request. Are you able to test it and confirms that it works for you, that would be most helpful!

#6194

@jamiemurray
Copy link
Author

jamiemurray commented Jul 24, 2018 via email

@jamiemurray
Copy link
Author

#6194 resolves the issue.

Dillon-Brown added a commit that referenced this issue Aug 30, 2018
Fixed #6131 - Issue with DetailView of Email module on 7.10.7 xampp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Emails Issues & PRs related to all things regarding emails & email module Priority:Critical Issues & PRs that are critical; broken core functionality, fatal errors - there are no workarounds Status:Fix Proposed A issue that has a PR related to it that provides a possible resolution Type:Bug Bugs within the core SuiteCRM codebase
Projects
None yet
Development

No branches or pull requests

4 participants