cancel
Showing results for 
Search instead for 
Did you mean: 

Error while patching SUPEE-8788 security patch (For Magento 1.7.0.2)

Error while patching SUPEE-8788 security patch (For Magento 1.7.0.2)

Error while patching SUPEE-8788 security patch (For Magento 1.7.0.2)
 

I'm facing error while applying the SUPEE-8788 security patch. The issue is,

 

checking file app/code/core/Mage/Customer/controllers/AccountController.php
Hunk #6 FAILED at 272.

 

From Patch file:

@@ -267,125 +272,254 @@ class Mage_Customer_AccountController extends Mage_Core_Controller_Front_Action
return;
}

+ /** @var $session Mage_Customer_Model_Session */
$session = $this->_getSession();
if ($session->isLoggedIn()) {
$this->_redirect('*/*/');
return;
}

- if ($this->getRequest()->isPost()) {
- $errors = array();
+ if (!$this->getRequest()->isPost()) {
+ $errUrl = $this->_getUrl('*/*/create', array('_secure' => true));
+ $this->_redirectError($errUrl);
+ return;

 

But, when I looking my file. it varies  - line 272,

Also, FYI, I installed following patches.
SUPEE -5345
SUPEE-1533
SUPEE-5994
SUPEE-6237
SUPEE-6285
SUPEE-6482
SUPEE-6788
SUPEE-1868
SUPEE-4334-v1.11.1.0
SUPEE-7616
SUPEE-7405
SUPEE-7405v1.1
SUPEE-8167
SUPEE-1049
SUPEE-8967
SUPEE-9652
SUPEE-3941.

picture :Untitled.png

Anyone, Please support to resolve this issue.

Thanks in Advance!! 

 

1 REPLY

Re: Error while patching SUPEE-8788 security patch (For Magento 1.7.0.2)

Hi Guys, 

I fixed above I mentioned the issue. But now I'm facing another one.

 

checking file app/code/core/Mage/Customer/controllers/AccountController.php
Hunk #6 FAILED at 272.
Hunk #7 succeeded at 410 (offset 2 lines).
Hunk #8 succeeded at 430 (offset 2 lines).
Hunk #9 succeeded at 442 (offset 2 lines).
Hunk #10 succeeded at 457 (offset 2 lines).
Hunk #11 succeeded at 481 (offset 2 lines).
Hunk #12 succeeded at 506 (offset 2 lines).
Hunk #13 succeeded at 527 (offset 2 lines).
Hunk #14 succeeded at 546 (offset 2 lines).
Hunk #15 succeeded at 588 (offset 2 lines).
Hunk #16 succeeded at 604 (offset 2 lines).
Hunk #17 succeeded at 651 (offset 2 lines).
Hunk #18 succeeded at 669 (offset 2 lines).
Hunk #19 succeeded at 707 (offset 2 lines).
Hunk #20 succeeded at 731 (offset 2 lines).
Hunk #21 succeeded at 764 (offset 2 lines).
Hunk #22 succeeded at 787 (offset 2 lines).
Hunk #23 succeeded at 808 (offset 2 lines).
Hunk #24 succeeded at 886 (offset 2 lines).
1 out of 24 hunks FAILED

 

For your kind information, Now the line looks like correct...But I'm not sure,  why its throwing error...

 

Code from Patch file:

@@ -267,125 +272,254 @@ class Mage_Customer_AccountController extends Mage_Core_Controller_Front_Action
return;
}

+ /** @var $session Mage_Customer_Model_Session */
$session = $this->_getSession();
if ($session->isLoggedIn()) {
$this->_redirect('*/*/');
return;
}

- if ($this->getRequest()->isPost()) {
- $errors = array();
+ if (!$this->getRequest()->isPost()) {
+ $errUrl = $this->_getUrl('*/*/create', array('_secure' => true));
+ $this->_redirectError($errUrl);
+ return;
+ }

 

Account Controller:

 

Any ideas what else I can try?