avatar andrepereiradasilva
andrepereiradasilva
19 Jun 2016

Steps to reproduce the issue

When apliyng a patch with a lot of files i get a memory exausted fatal error.

This is the patch i'm applying infograf768/joomla-cms#30

Expected result

Patch applied.

Actual result

Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 4078152 bytes) in /path/to/joomla-staging/libraries/joomla/database/driver.php on line 2041

System information (as much as possible)

Joomla 3.6.0 beta 2
PHP memory limit 256M.

Additional comments

I know i can put the php memory limit higher.

But this is the memory limit i used for joomla forever and always worked.
So why is the patchtester using so much memory.

avatar andrepereiradasilva andrepereiradasilva - open - 19 Jun 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Jun 2016

just tested with 350MB and still get the fatal error
Fatal error: Allowed memory size of 367001600 bytes exhausted (tried to allocate 4078152 bytes) in /path/to/joomla-staging/libraries/joomla/database/driver.php on line 2041

avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Jun 2016

Aditional info.
If i remove this lines https://github.com/joomla-extensions/patchtester/blob/master/administrator/components/com_patchtester/PatchTester/Model/PullModel.php#L297-L309 it applies with no memory issues (of course it doesn't save as applied).

So the problem should be there.

avatar mbabker
mbabker - comment - 19 Jun 2016

I removed JTable completely. So we'll see how much that helps.

avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Jun 2016

ok thanks!

avatar mbabker
mbabker - comment - 25 Jun 2016

Have you had a chance to test perhaps? Between this and other changes I made today, a lot of moving pieces in the code right now.

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Jun 2016

i tested the same patch and worked now. thanks.

avatar andrepereiradasilva andrepereiradasilva - close - 28 Jun 2016
avatar andrepereiradasilva andrepereiradasilva - change - 28 Jun 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-06-28 10:56:51
Closed_By andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 28 Jun 2016

Add a Comment

Login with GitHub to post a comment