When max_execution_time is 0, no time limit is imposed. However, Backup and Migrate treats this as 0 seconds. The drupal_set_time_limit() wrapper properly recognizes when the max_execution_time is unlimited, so it should be used instead of set_time_limit().

CommentFileSizeAuthor
#2 max_execution_time-2940451-1.patch1.65 KBdarren oh

Comments

Darren Oh created an issue. See original summary.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new max_execution_time-2940451-1.patch1.65 KB
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Assigned: darren oh » Unassigned
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Active » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #2939266: Plan for Backup and Migrate 7.x-3.5

Thanks Darren!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed. Thanks.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI I released 3.5 today with this fix. Thank you again!

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

Thank you, Damien. Good to be able to work with you.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Likewise :) I hope you are well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.