getItemById($itemId) for Magento Quote Model not working as expected in Magento 2

Here is my code, where I'm calling Quote model file and getItemById function.

 public function __construct(

        MagentoQuoteModelQuote $quote

    ) {

        $this->quote = $quote;

    }



$itemId = "Quote Item ID";

$data = $this->quote->getItemById($itemId);

Above code returns FALSE - even there are 65 Quote Items are exists!

So I checked the Magento core method of getItemById in -

/vendor/magento/module-quote/Model/Quote.php

     /**

     * Retrieve item model object by item identifier

     *

     * @param   int $itemId

     * @return  MagentoQuoteModelQuoteItem|false

     */

    public function getItemById($itemId)

    {

        foreach ($this->getItemsCollection() as $item) {

            if ($item->getId() == $itemId) {

                return $item;

            }

        }



        return false;

    }

If I print the $this->getItemsCollection()->getData() then it returns the 65 Item arrays.

But As Magento passing directly the collection object $this->getItemsCollection() in foreach

So It's not going in-side the foreach loop which is expected result.

Is it a Magento bug or I am doing something wrong ?

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

Do you have quote id?
– Rohan Hapani
4 hours ago

1

No, I have Quote Item ID
– Aditya Shah
4 hours ago

add a comment |

Here is my code, where I'm calling Quote model file and getItemById function.

 public function __construct(

        MagentoQuoteModelQuote $quote

    ) {

        $this->quote = $quote;

    }



$itemId = "Quote Item ID";

$data = $this->quote->getItemById($itemId);

Above code returns FALSE - even there are 65 Quote Items are exists!

So I checked the Magento core method of getItemById in -

/vendor/magento/module-quote/Model/Quote.php

     /**

     * Retrieve item model object by item identifier

     *

     * @param   int $itemId

     * @return  MagentoQuoteModelQuoteItem|false

     */

    public function getItemById($itemId)

    {

        foreach ($this->getItemsCollection() as $item) {

            if ($item->getId() == $itemId) {

                return $item;

            }

        }



        return false;

    }

If I print the $this->getItemsCollection()->getData() then it returns the 65 Item arrays.

But As Magento passing directly the collection object $this->getItemsCollection() in foreach

So It's not going in-side the foreach loop which is expected result.

Is it a Magento bug or I am doing something wrong ?

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

Do you have quote id?
– Rohan Hapani
4 hours ago

1

No, I have Quote Item ID
– Aditya Shah
4 hours ago

add a comment |

Here is my code, where I'm calling Quote model file and getItemById function.

 public function __construct(

        MagentoQuoteModelQuote $quote

    ) {

        $this->quote = $quote;

    }



$itemId = "Quote Item ID";

$data = $this->quote->getItemById($itemId);

Above code returns FALSE - even there are 65 Quote Items are exists!

So I checked the Magento core method of getItemById in -

/vendor/magento/module-quote/Model/Quote.php

     /**

     * Retrieve item model object by item identifier

     *

     * @param   int $itemId

     * @return  MagentoQuoteModelQuoteItem|false

     */

    public function getItemById($itemId)

    {

        foreach ($this->getItemsCollection() as $item) {

            if ($item->getId() == $itemId) {

                return $item;

            }

        }



        return false;

    }

If I print the $this->getItemsCollection()->getData() then it returns the 65 Item arrays.

But As Magento passing directly the collection object $this->getItemsCollection() in foreach

So It's not going in-side the foreach loop which is expected result.

Is it a Magento bug or I am doing something wrong ?

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

Here is my code, where I'm calling Quote model file and getItemById function.

 public function __construct(

        MagentoQuoteModelQuote $quote

    ) {

        $this->quote = $quote;

    }



$itemId = "Quote Item ID";

$data = $this->quote->getItemById($itemId);

Above code returns FALSE - even there are 65 Quote Items are exists!

So I checked the Magento core method of getItemById in -

/vendor/magento/module-quote/Model/Quote.php

     /**

     * Retrieve item model object by item identifier

     *

     * @param   int $itemId

     * @return  MagentoQuoteModelQuoteItem|false

     */

    public function getItemById($itemId)

    {

        foreach ($this->getItemsCollection() as $item) {

            if ($item->getId() == $itemId) {

                return $item;

            }

        }



        return false;

    }

If I print the $this->getItemsCollection()->getData() then it returns the 65 Item arrays.

But As Magento passing directly the collection object $this->getItemsCollection() in foreach

So It's not going in-side the foreach loop which is expected result.

Is it a Magento bug or I am doing something wrong ?

magento2 model quote bug quoteitem

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

edited 4 hours ago

asked 4 hours ago

Aditya Shah

3,4852834

asked 4 hours ago

Aditya Shah

3,4852834

asked 4 hours ago

Aditya Shah

3,4852834

Do you have quote id?
– Rohan Hapani
4 hours ago

1

No, I have Quote Item ID
– Aditya Shah
4 hours ago

add a comment |

Do you have quote id?
– Rohan Hapani
4 hours ago

1

No, I have Quote Item ID
– Aditya Shah
4 hours ago

Do you have quote id?
– Rohan Hapani
4 hours ago

No, I have Quote Item ID
– Aditya Shah
4 hours ago

add a comment |

2 Answers
2

active

oldest

votes

Try to use this below code :

/**

 * @var MagentoQuoteModelQuoteItem

 */

protected $_itemModel;



public function __construct(

    .........

    MagentoQuoteModelQuoteItem $itemModel

    .........

)

{

    .........

    $this->_itemModel = $itemModel;

    .........

}



public function yourFunction()

{

    $itemId = "Quote Item ID";

    $this->_itemModel->load($itemId);

}

answered 4 hours ago

Rohan Hapani

5,81921662

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

|
show 1 more comment

Instead of injecting MagentoQuoteModelQuote class in constructor, try injecting MagentoCheckoutModelSession and then

$quote = $this->_session->getQuote();

This assumes your code is working with a frontend active checkout session

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

|
show 2 more comments

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');
});

Post as a guest

Name

Required, but never shown

StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f256754%2fgetitembyiditemid-for-magento-quote-model-not-working-as-expected-in-magento%23new-answer', 'question_page');
}
);

Post as a guest

Name

Required, but never shown

2 Answers
2

active

oldest

votes

2 Answers
2

active

oldest

votes

Try to use this below code :

/**

 * @var MagentoQuoteModelQuoteItem

 */

protected $_itemModel;



public function __construct(

    .........

    MagentoQuoteModelQuoteItem $itemModel

    .........

)

{

    .........

    $this->_itemModel = $itemModel;

    .........

}



public function yourFunction()

{

    $itemId = "Quote Item ID";

    $this->_itemModel->load($itemId);

}

answered 4 hours ago

Rohan Hapani

5,81921662

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

|
show 1 more comment

Try to use this below code :

/**

 * @var MagentoQuoteModelQuoteItem

 */

protected $_itemModel;



public function __construct(

    .........

    MagentoQuoteModelQuoteItem $itemModel

    .........

)

{

    .........

    $this->_itemModel = $itemModel;

    .........

}



public function yourFunction()

{

    $itemId = "Quote Item ID";

    $this->_itemModel->load($itemId);

}

answered 4 hours ago

Rohan Hapani

5,81921662

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

|
show 1 more comment

Try to use this below code :

/**

 * @var MagentoQuoteModelQuoteItem

 */

protected $_itemModel;



public function __construct(

    .........

    MagentoQuoteModelQuoteItem $itemModel

    .........

)

{

    .........

    $this->_itemModel = $itemModel;

    .........

}



public function yourFunction()

{

    $itemId = "Quote Item ID";

    $this->_itemModel->load($itemId);

}

answered 4 hours ago

Rohan Hapani

5,81921662

Try to use this below code :

/**

 * @var MagentoQuoteModelQuoteItem

 */

protected $_itemModel;



public function __construct(

    .........

    MagentoQuoteModelQuoteItem $itemModel

    .........

)

{

    .........

    $this->_itemModel = $itemModel;

    .........

}



public function yourFunction()

{

    $itemId = "Quote Item ID";

    $this->_itemModel->load($itemId);

}

answered 4 hours ago

Rohan Hapani

5,81921662

answered 4 hours ago

Rohan Hapani

5,81921662

answered 4 hours ago

Rohan Hapani

5,81921662

answered 4 hours ago

Rohan Hapani

5,81921662

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

|
show 1 more comment

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

Worked like a charm! Thanks bro :)
– Aditya Shah
3 hours ago

But still we need to check Why Magento's Quote model function is not working!
– Aditya Shah
3 hours ago

I suppose you are not loading the right Quote object (it is normal if you don't have quote_id)
– Raul Sanchez
3 hours ago

No, It is. Please check the /vendor/magento/module-quote/Model/Quote.php file.
– Aditya Shah
3 hours ago

I mean that $this->quote in your code. It is just the injected object in constructor? without loading any id? If that's the case, that object will provide the method, but it won't content any item
– Raul Sanchez
3 hours ago

|
show 1 more comment

Instead of injecting MagentoQuoteModelQuote class in constructor, try injecting MagentoCheckoutModelSession and then

$quote = $this->_session->getQuote();

This assumes your code is working with a frontend active checkout session

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

|
show 2 more comments

Instead of injecting MagentoQuoteModelQuote class in constructor, try injecting MagentoCheckoutModelSession and then

$quote = $this->_session->getQuote();

This assumes your code is working with a frontend active checkout session

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

|
show 2 more comments

Instead of injecting MagentoQuoteModelQuote class in constructor, try injecting MagentoCheckoutModelSession and then

$quote = $this->_session->getQuote();

This assumes your code is working with a frontend active checkout session

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

Instead of injecting MagentoQuoteModelQuote class in constructor, try injecting MagentoCheckoutModelSession and then

$quote = $this->_session->getQuote();

This assumes your code is working with a frontend active checkout session

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

edited 3 hours ago

answered 4 hours ago

Raul Sanchez

1,82231135

answered 4 hours ago

Raul Sanchez

1,82231135

answered 4 hours ago

Raul Sanchez

1,82231135

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

|
show 2 more comments

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

Let me check @Raul
– Aditya Shah
4 hours ago

Not working, returning false
– Aditya Shah
4 hours ago

mmm I assume you are using an existing item_id instead of "Quote Item ID" text, right?
– Raul Sanchez
4 hours ago

Yes yes of-course
– Aditya Shah
4 hours ago

I have updated the answer
– Raul Sanchez
3 hours ago

|
show 2 more comments

draft saved

draft discarded

Thanks for contributing an answer to Magento Stack Exchange!

Please be sure to answer the question. Provide details and share your research!

But avoid …

Asking for help, clarification, or responding to other answers.

Making statements based on opinion; back them up with references or personal experience.

To learn more, see our tips on writing great answers.

Some of your past answers have not been well-received, and you're in danger of being blocked from answering.

Please pay close attention to the following guidance:

Please be sure to answer the question. Provide details and share your research!

But avoid …

Asking for help, clarification, or responding to other answers.

Making statements based on opinion; back them up with references or personal experience.

To learn more, see our tips on writing great answers.

draft saved

draft discarded

Sign up or log in

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

Post as a guest

Name

Required, but never shown

Post as a guest

Name

Required, but never shown

Sign up or log in

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

Post as a guest

Name

Required, but never shown

Sign up or log in

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

Post as a guest

Name

Required, but never shown

Sign up or log in

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

Post as a guest

Name

Required, but never shown

Name

Required, but never shown

Name

Required, but never shown

This page is only for reference, If you need detailed information, please check here

搜尋此網誌

Ykhjuy