0000025: Onepage Checkout: Ajax issue

(0000031)
w3design (zgłaszający)
2009-06-11 14:30

Hi,

we encountered the same problem and (after hours of debugging) could provide a bugfix.

You have to edit a line of php code in the following file:

./app/code/local/Mage/Checkout/controllers/OnepageController.php

in the method saveShippingMethodAction().
There you have to make this modification:

public function saveShippingMethodAction()
    {
        $this->_expireAjax();
        if ($this->getRequest()->isPost()) {
            $data = $this->getRequest()->getPost('shipping_method', '');
            $result = $this->getOnepage()->saveShippingMethod($data);
            /*
            $result will have erro data if shipping method is empty
            */
            if(!$result) {
                Mage::dispatchEvent('checkout_controller_onepage_save_shipping_method', array('request'=>$this->getRequest(), 'quote'=>$this->getOnepage()->getQuote()));
                // original place
                // $this->getResponse()->setBody(Zend_Json::encode($result));

                $result['goto_section'] = 'payment';
                $result['update_section'] = array(
                    'name' => 'payment-method',
                    'html' => $this->_getPaymentMethodsHtml()
                );

                // new (and better) place
                // see /app/code/community/Flagbit/Typo3connect/Controller/Response.php :: setBody() / ajaxHandler()

                $this->getResponse()->setBody(Zend_Json::encode($result));

// $result['payment_methods_html'] = $this->_getPaymentMethodsHtml();
            }
            $this->getResponse()->setBody(Zend_Json::encode($result));
        }

    }

This change fixed the problem for us.

Kind regards,

Marco Stoll
w3design. GmbH
marco.stoll@w3design.de

(0000032)
w3design (zgłaszający)
2009-06-11 14:32

p.s.: we already filed an official bug issue on http://www.magentocommerce.com/bug-tracking [^]

(0000033)
dschwarz (zgłaszający)
2009-06-11 15:00
edytowano: 2009-06-11 15:00

thank you, this change fixed the problem!

i think this bug can be closed here now, because its a magento-core issue.

(0000070)
drlrdsen (administrator)
2009-10-05 15:04

@w3design: Did they add you code to the core?

(0000078)
w3design (zgłaszający)
2009-10-05 15:14

I did not get any feedback until now. But I've not checked any current magento version for changes myself. The magento version used in this project was v1.3.1.

I filed the issue here: http://www.magentocommerce.com/bug-tracking/issue?issue=6570 [^]

(0000079)
drlrdsen (administrator)
2009-10-05 15:14

Okay, thanks anyways, we'll check for it as soon as possible.

Komentarze
(0000031) w3design (zgłaszający) 2009-06-11 14:30	Hi, we encountered the same problem and (after hours of debugging) could provide a bugfix. You have to edit a line of php code in the following file: ./app/code/local/Mage/Checkout/controllers/OnepageController.php in the method saveShippingMethodAction(). There you have to make this modification: public function saveShippingMethodAction() { $this->_expireAjax(); if ($this->getRequest()->isPost()) { $data = $this->getRequest()->getPost('shipping_method', ''); $result = $this->getOnepage()->saveShippingMethod($data); /* $result will have erro data if shipping method is empty */ if(!$result) { Mage::dispatchEvent('checkout_controller_onepage_save_shipping_method', array('request'=>$this->getRequest(), 'quote'=>$this->getOnepage()->getQuote())); // original place // $this->getResponse()->setBody(Zend_Json::encode($result)); $result['goto_section'] = 'payment'; $result['update_section'] = array( 'name' => 'payment-method', 'html' => $this->_getPaymentMethodsHtml() ); // new (and better) place // see /app/code/community/Flagbit/Typo3connect/Controller/Response.php :: setBody() / ajaxHandler() $this->getResponse()->setBody(Zend_Json::encode($result)); // $result['payment_methods_html'] = $this->_getPaymentMethodsHtml(); } $this->getResponse()->setBody(Zend_Json::encode($result)); } } This change fixed the problem for us. Kind regards, Marco Stoll w3design. GmbH marco.stoll@w3design.de

(0000032) w3design (zgłaszający) 2009-06-11 14:32	p.s.: we already filed an official bug issue on http://www.magentocommerce.com/bug-tracking [^]

(0000033) dschwarz (zgłaszający) 2009-06-11 15:00 edytowano: 2009-06-11 15:00	thank you, this change fixed the problem! i think this bug can be closed here now, because its a magento-core issue.

(0000070) drlrdsen (administrator) 2009-10-05 15:04	@w3design: Did they add you code to the core?

(0000078) w3design (zgłaszający) 2009-10-05 15:14	I did not get any feedback until now. But I've not checked any current magento version for changes myself. The magento version used in this project was v1.3.1. I filed the issue here: http://www.magentocommerce.com/bug-tracking/issue?issue=6570 [^]

(0000079) drlrdsen (administrator) 2009-10-05 15:14	Okay, thanks anyways, we'll check for it as soon as possible.

Historia zagadnienia
Data modyfikacji	Nazwa użytkownika	Pole	Zmiana
2009-06-09 13:02	dschwarz	Nowe Zagadnienie
2009-06-11 14:30	w3design	Dodano komentarz: 0000031
2009-06-11 14:32	w3design	Dodano komentarz: 0000032
2009-06-11 15:00	dschwarz	Dodano komentarz: 0000033
2009-06-11 15:00	dschwarz	Edycja komentarza: 0000033
2009-10-05 15:04	drlrdsen	Dodano komentarz: 0000070
2009-10-05 15:04	drlrdsen	Stan	nowy => zwrócony
2009-10-05 15:14	w3design	Dodano komentarz: 0000078
2009-10-05 15:14	drlrdsen	Dodano komentarz: 0000079

Powiązania