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

[raja7]

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 :

Anyone, Please support to resolve this issue.

Thanks in Advance!! 

[raja7]

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 :

Anyone, Please support to resolve this issue.

Thanks in Advance!! 

[raja7]

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?

[jevinsu]

Similar error at the same file for this patch on ver 1.6.0, as following:

patching file app/code/core/Mage/Customer/controllers/AccountController.php

Hunk #1 succeeded at 124 (offset -10 lines).

Hunk #2 succeeded at 146 (offset -10 lines).

Hunk #3 succeeded at 178 (offset -10 lines).

Hunk #4 succeeded at 186 (offset -10 lines).

Hunk #5 succeeded at 195 (offset -10 lines).

Hunk #6 FAILED at 255.

Hunk #7 succeeded at 521 (offset -14 lines).

Hunk #8 succeeded at 533 (offset -14 lines).

Hunk #9 succeeded at 548 (offset -14 lines).

Hunk #10 succeeded at 572 (offset -14 lines).

Hunk #11 succeeded at 597 (offset -14 lines).

Hunk #12 succeeded at 618 (offset -14 lines).

Hunk #13 succeeded at 637 (offset -14 lines).

Hunk #14 succeeded at 677 (offset -14 lines).

Hunk #15 succeeded at 726 (offset -14 lines).

Hunk #16 succeeded at 749 (offset -14 lines).

Hunk #17 succeeded at 770 (offset -14 lines).

1 out of 17 hunks FAILED -- saving rejects to file app/code/core/Mage/Customer/controllers/AccountController.php.rej

[Mukesh Tiwari]

Hi @raja7

Check following :

1) Check if your core file is modified. If yes then revert with correct file from fresh download of you Magento version then try again.

2) Did you apply SUPEE-6788 and SUPEE-7405 before applying SUPEE-8788 patch.