Gsyfuy

What does convergence in distribution "in the Gromov–Hausdorff" sense mean?

How do I reset passwords on multiple websites easily?

If Nick Fury and Coulson already knew about aliens (Kree and Skrull) why did they wait until Thor's appearance to start making weapons?

What was the first Unix version to run on a microcomputer?

"In the right combination" vs "with the right combination"?

How powerful is the invisibility granted by the Gloom Stalker ranger's Umbral Sight feature?

Won the lottery - how do I keep the money?

Is it ever safe to open a suspicious html file (e.g. email attachment)?

Written every which way

How to safely derail a train during transit?

Can I equip Skullclamp on a creature I am sacrificing?

Does it take more energy to get to Venus or to Mars?

Why is the US ranked as #45 in Press Freedom ratings, despite its extremely permissive free speech laws?

Why do professional authors make "consistency" mistakes? And how to avoid them?

How did people program for Consoles with multiple CPUs?

Why am I allowed to create multiple unique pointers from a single object?

How to start emacs in "nothing" mode (`fundamental-mode`)

Can we say or write : "No, it'sn't"?

Is it possible to search for a directory/file combination?

Is "for causing autism in X" grammatical?

What does "Its cash flow is deeply negative" mean?

How to count occurrences of text in a file?

Should I tutor a student who I know has cheated on their homework?

Is there an analogue of projective spaces for proper schemes?

Resolved - Mass Action - Mass Database Update in Magento Grid

0

I have this grid developed for the custom module. I added the mass delete action which works fine. But the mass Approve/Disapprove actions work only for one value, when I select multiple values I get this error.
Please help to resolve this:

Here is the code for the approve function in controller file:

    public function massApproveAction() {

       $requestIds = $this->getRequest()->getParam('id');

       if(!is_array($requestIds)) {

       Mage::getSingleton('adminhtml/session')->addError(Mage::helper('adminhtml')->__('Please select reqeust(s)'));

       } else { 

       try {    



        foreach ($requestIds as $requestId) {

            $RequestData = Mage::getModel('cpstest_productcomment/cps')->load($requestId); 

            $id = $this->getRequest()->getParam('id');

            $data = array('comment_status'=>'Approved');

            $model = Mage::getModel('cpstest_productcomment/cps')->addData($data);      

            }

                $model->setId($id)->save();

                echo "Data updated successfully.";



        } catch (Exception $e){

                echo $e->getMessage(); 

        }

    }

}

magento-1.9 module grid custom massaction

share|improve this question

edited Apr 6 '17 at 18:22

bestwebdevs

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

add a comment | 

0

I have this grid developed for the custom module. I added the mass delete action which works fine. But the mass Approve/Disapprove actions work only for one value, when I select multiple values I get this error.
Please help to resolve this:

Here is the code for the approve function in controller file:

    public function massApproveAction() {

       $requestIds = $this->getRequest()->getParam('id');

       if(!is_array($requestIds)) {

       Mage::getSingleton('adminhtml/session')->addError(Mage::helper('adminhtml')->__('Please select reqeust(s)'));

       } else { 

       try {    



        foreach ($requestIds as $requestId) {

            $RequestData = Mage::getModel('cpstest_productcomment/cps')->load($requestId); 

            $id = $this->getRequest()->getParam('id');

            $data = array('comment_status'=>'Approved');

            $model = Mage::getModel('cpstest_productcomment/cps')->addData($data);      

            }

                $model->setId($id)->save();

                echo "Data updated successfully.";



        } catch (Exception $e){

                echo $e->getMessage(); 

        }

    }

}

magento-1.9 module grid custom massaction

share|improve this question

edited Apr 6 '17 at 18:22

bestwebdevs

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

add a comment | 

I have this grid developed for the custom module. I added the mass delete action which works fine. But the mass Approve/Disapprove actions work only for one value, when I select multiple values I get this error.
Please help to resolve this:

Here is the code for the approve function in controller file:

    public function massApproveAction() {

       $requestIds = $this->getRequest()->getParam('id');

       if(!is_array($requestIds)) {

       Mage::getSingleton('adminhtml/session')->addError(Mage::helper('adminhtml')->__('Please select reqeust(s)'));

       } else { 

       try {    



        foreach ($requestIds as $requestId) {

            $RequestData = Mage::getModel('cpstest_productcomment/cps')->load($requestId); 

            $id = $this->getRequest()->getParam('id');

            $data = array('comment_status'=>'Approved');

            $model = Mage::getModel('cpstest_productcomment/cps')->addData($data);      

            }

                $model->setId($id)->save();

                echo "Data updated successfully.";



        } catch (Exception $e){

                echo $e->getMessage(); 

        }

    }

}

magento-1.9 module grid custom massaction

share|improve this question

edited Apr 6 '17 at 18:22

bestwebdevs

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

    public function massApproveAction() {

       $requestIds = $this->getRequest()->getParam('id');

       if(!is_array($requestIds)) {

       Mage::getSingleton('adminhtml/session')->addError(Mage::helper('adminhtml')->__('Please select reqeust(s)'));

       } else { 

       try {    



        foreach ($requestIds as $requestId) {

            $RequestData = Mage::getModel('cpstest_productcomment/cps')->load($requestId); 

            $id = $this->getRequest()->getParam('id');

            $data = array('comment_status'=>'Approved');

            $model = Mage::getModel('cpstest_productcomment/cps')->addData($data);      

            }

                $model->setId($id)->save();

                echo "Data updated successfully.";



        } catch (Exception $e){

                echo $e->getMessage(); 

        }

    }

}

share|improve this question

edited Apr 6 '17 at 18:22

bestwebdevs

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

share|improve this question

edited Apr 6 '17 at 18:22

bestwebdevs

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

asked Apr 6 '17 at 0:24

bestwebdevsbestwebdevs

122213

1 Answer
1

active

oldest

votes

1

Everything is correct but at the time of saving data you set all request params in setId method. Please find below line and correct it.

$model->setId($id)->save();

Replace with below code

$model->setId($requestId)->save();

share|improve this answer

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Thanks for your respond. With this code I don't get the error any more, but it still saves only one value.
  
  – bestwebdevs
  Apr 6 '17 at 16:58
  
  
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Your solution is correct, I made a mistake and put it outside the loop, that's why it was only saving 1 value. Then I put it in the foreach loop and it works great :) Thank you!
  
  – bestwebdevs
  Apr 6 '17 at 17:34
  

  
  
  
  

add a comment | 

Your Answer

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "479"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f167880%2fresolved-mass-action-mass-database-update-in-magento-grid%23new-answer', 'question_page');
}
);

Post as a guest

Name

Email

Required, but never shown

1

Everything is correct but at the time of saving data you set all request params in setId method. Please find below line and correct it.

$model->setId($id)->save();

Replace with below code

$model->setId($requestId)->save();

share|improve this answer

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Thanks for your respond. With this code I don't get the error any more, but it still saves only one value.
  
  – bestwebdevs
  Apr 6 '17 at 16:58
  
  
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Your solution is correct, I made a mistake and put it outside the loop, that's why it was only saving 1 value. Then I put it in the foreach loop and it works great :) Thank you!
  
  – bestwebdevs
  Apr 6 '17 at 17:34
  

  
  
  
  

add a comment | 

1

Everything is correct but at the time of saving data you set all request params in setId method. Please find below line and correct it.

$model->setId($id)->save();

Replace with below code

$model->setId($requestId)->save();

share|improve this answer

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Thanks for your respond. With this code I don't get the error any more, but it still saves only one value.
  
  – bestwebdevs
  Apr 6 '17 at 16:58
  
  
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  
  Your solution is correct, I made a mistake and put it outside the loop, that's why it was only saving 1 value. Then I put it in the foreach loop and it works great :) Thank you!
  
  – bestwebdevs
  Apr 6 '17 at 17:34
  

  
  
  
  

add a comment | 

Everything is correct but at the time of saving data you set all request params in setId method. Please find below line and correct it.

$model->setId($id)->save();

Replace with below code

$model->setId($requestId)->save();

share|improve this answer

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

$model->setId($id)->save();

$model->setId($requestId)->save();

share|improve this answer

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

edited 8 mins ago

Teja Bhagavan Kollepara

3,01241949

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418

answered Apr 6 '17 at 6:25

ARVIND KARKARARVIND KARKAR

4722418