Skip to content

This will fix the low stock report error#8

Open
captainyoung wants to merge 3 commits intoetailer:masterfrom
captainyoung:master
Open

This will fix the low stock report error#8
captainyoung wants to merge 3 commits intoetailer:masterfrom
captainyoung:master

Conversation

@captainyoung
Copy link
Copy Markdown
Contributor

No description provided.

This will fix the low stock report.
Override low stock collection
*/
public function useNotifyStockQtyFilter($storeId = null)
{
Mage::log('My log entry');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, forgot to remove that. You can remove.
On Jun 27, 2016 13:16, "Robert Coleman" notifications@github.com wrote:

In
app/code/community/Etailer/Backorders/Model/Reports/Resource/Product/Lowstock/Collection.php
#8 (comment)
:

  • * @category Mage
  • * @Package Mage_Reports
  • * @author Magento Core Team core@magentocommerce.com
  • */
    +class Etailer_Backorders_Model_Reports_Resource_Product_Lowstock_Collection extends Mage_Reports_Model_Resource_Product_Lowstock_Collection
    +{
  • /**
  • \* Add Notify Stock Qty Condition to collection
    
  • *
    
  • \* @param int $storeId
    
  • \* @return Mage_Reports_Model_Resource_Product_Lowstock_Collection
    
  • */
    
  • public function useNotifyStockQtyFilter($storeId = null)
  • {
  •   Mage::log('My log entry');
    

why is this here?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/etailer/Etailer_Backorders/pull/8/files/378fb99a414b21b607e74f0a815898e82ac9a36d#r68673440,
or mute the thread
https://github.com/notifications/unsubscribe/AQaVeFrYVBQRRx1aAFvrCpoQxZX0IkWeks5qQFnWgaJpZM4I-NJe
.

@seansan
Copy link
Copy Markdown

seansan commented Jan 17, 2017

Is this to be merged now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants