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

7.11.9 Release #8141

Merged
merged 541 commits into from
Nov 4, 2019
Merged

7.11.9 Release #8141

merged 541 commits into from
Nov 4, 2019

Conversation

Dillon-Brown
Copy link
Contributor

@Dillon-Brown Dillon-Brown commented Oct 28, 2019

Description

Motivation and Context

PHP Fatal error:  Uncaught RuntimeException: Directory "" was not created in /var/www/html/suitecrm-mysql/modules/InboundEmail/InboundEmail.php:4614
Stack trace:
#0 /var/www/html/suitecrm-mysql/modules/InboundEmail/InboundEmail.php(4514): InboundEmail->saveAttachmentBinaries(Object(Note), 109, '1.2', Object(stdClass), false)
#1 /var/www/html/suitecrm-mysql/modules/InboundEmail/InboundEmail.php(4444): InboundEmail->saveAttachments(109, Array, '71fe231b-5ea9-f...', 1, false)
#2 /var/www/html/suitecrm-mysql/modules/InboundEmail/InboundEmail.php(5314): InboundEmail->saveAttachments(109, Array, '71fe231b-5ea9-f...', '0', false)
#3 /var/www/html/suitecrm-mysql/modules/Schedulers/_AddJobsHere.php(636): InboundEmail->returnImportedEmail(109, 365, false, true, true)
#4 /var/www/html/suitecrm-mysql/modules/SchedulersJobs/SchedulersJob.php(540): pollMonitoredInboxesAOP(Object(SchedulersJob))
#5 /var/www/html/suitecrm-mysql/include/SugarQueue/SugarCronJobs.php(191): SchedulersJob->runJob()
#6 /var/www/html/suitecrm-mysql/include/SugarQueue/SugarCronJobs.php(231): SugarCronJobs->executeJob(Object(SchedulersJob in /var/www/html/suitecrm-mysql/modules/InboundEmail/InboundEmail.php on line 4614

How To Test This

  1. Setup group inbound email.
  2. Enable email -> cases.
  3. If cron is not already setup, add root user to config allowed_cron_users and run php -f cron.php after sending a few emails to the group account.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

  • My code follows the code style of this project found here.
  • My change requires a change to the documentation.
  • I have read the How to Contribute guidelines.

Dillon-Brown and others added 30 commits September 30, 2019 11:07
# Conflicts:
#	.codecov.yml
#	codeception.yml
#	composer.lock
#	modules/Import/Importer.php
The bean is cached anyway. I can't seem to get around that.
PHP 5 is braindead. Type hinting a string will cause an error
like this:
"Argument 1 passed to GoogleSyncHelper::wipeLocalSyncData() must be an
instance of string, string given"
... braindead...
These changes fix issue salesagility#7926

The OAuth clients/tokens page has no widths defined, resulting in the totalWidth being 0, and so does the adjustment. The quickfix is to prevent dividing by the variable in such case, avoiding division by 0. When it does, we just set the final value to 0.
This change updates the .gitattributes to include several files in the
list of files and directories to ignore when creating an archive. This
allows us to download a ZIP of the repository without getting unncessary
configuration files, CI related files, github specific files, etc.
This prevents the travis.php.ini file from being included in archives
as it was missing the export-ignore flag. I've also fixed the
php_cs.xml file as it was missing the .xml file extension.
This prevents the unified_search_modules_display file from being
included in archives as it was missing the export-ignore flag.
…ty-dividebyzero

Hotfix: Do not divide by adjustment if it equals 0
craigpanton and others added 5 commits October 24, 2019 11:41
Fixed salesagility#1504 - check deleted flag on email_addresses and related bean
Compiling css and js files ahead of latest release.
Merging hotfix-7.10.x into hotfix and recompiling in preparation for latest release.
Copy link
Contributor

@craigpanton craigpanton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Dillon-Brown Dillon-Brown changed the base branch from hotfix-7.10.x to master October 31, 2019 09:55
@Mac-Rae
Copy link
Contributor

Mac-Rae commented Oct 31, 2019

Assessed 👍

@Mac-Rae Mac-Rae added the Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member label Oct 31, 2019
Mac-Rae and others added 2 commits October 31, 2019 10:12
v7.11.9-cf

# Conflicts:
#	modules/FP_events/FP_events.php
@Dillon-Brown Dillon-Brown changed the title Fixed missing cache directory throwing RunTimeException 7.11.9 Release Oct 31, 2019
@Dillon-Brown Dillon-Brown removed the Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member label Oct 31, 2019
'LBL_MY_EMAILS' => 'Emails',
'LBL_NEW_FORM_TITLE' => 'Archive Email',
'LBL_NONE' => 'None',
'LBL_NOT_SENT' => 'Send Error',
'LBL_NOTES_SUBPANEL_TITLE' => 'Attachments',
'LBL_NOTES_SUBPANEL_TITLE' => 'Attachments','LBL_MODULE_TITLE' => 'Emails:',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error here! @Dillon-Brown

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! removed.

@craigpanton craigpanton merged commit c9d2132 into salesagility:master Nov 4, 2019
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.