Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

M2.1 Marketing >> Reviews >> New Review - Sorting & Filters not functioning. #5391

Closed
praful-rajput opened this issue Jul 1, 2016 · 8 comments
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@praful-rajput
Copy link

Steps to reproduce

  1. Install Magento from "Full Release with Sample Data (ZIP with sample data)" from official page.
  2. Logged into admin panel
  3. Go to Marketing >> Reviews
  4. Click on "New Review" button.
  5. It's opening product list grid to select and review it.
  6. Grid sorting & Filtering is not working.

Expected result

  1. Sorting & Filter should be work over there.

Actual result

1.Sorting & Filtering is not working.

@pboisvert
Copy link

Not able to reproduce this issue on 2.1 build.

Screenshot of Marketing > Reviews with default sort on created
reviews-original

Screenshot of Marketing > Reviews with sort by Nickname (shows different order--now alphabetical)
review-sorted-nickname

Screenshot of Marketing > Reviews using Filter with keyword on Reviews column
reviews-filtered

Was this a clean install or an upgrade from previous version? Any other info to provide to help track down the issue?

@praful-rajput
Copy link
Author

praful-rajput commented Jul 2, 2016

It was clean install.

Please do not hesitate to reach out with any additional questions or
concerns that you might have.

Regards,

Praful Rajput

On Fri, Jul 1, 2016 at 11:26 PM, Paul Boisvert [email protected]
wrote:

Not able to reproduce this issue on 2.1 build.

Screenshot of Marketing > Reviews with default sort on created
[image: reviews-original]
https://cloud.githubusercontent.com/assets/5807369/16529851/f858598c-3f79-11e6-8f9f-54a1a918459b.png

Screenshot of Marketing > Reviews with sort by Nickname (shows different
order--now alphabetical)
[image: review-sorted-nickname]
https://cloud.githubusercontent.com/assets/5807369/16529859/00f195cc-3f7a-11e6-82dd-38287707d61a.png

Screenshot of Marketing > Reviews using Filter with keyword on Reviews
column
[image: reviews-filtered]
https://cloud.githubusercontent.com/assets/5807369/16529862/0653fef6-3f7a-11e6-8181-ccef106e2894.png

Was this a clean install or an upgrade from previous version? Any other
info to provide to help track down the issue?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#5391 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAnmm8xodlnDHErxEFOZ_AEY39SEQ5lMks5qRVTogaJpZM4JC1ri
.

@andimov andimov self-assigned this Aug 4, 2016
@andimov
Copy link
Contributor

andimov commented Aug 10, 2016

@vasuscoin
Thank you for reporting!
I have reproduced it and created an internal ticket for this issue - MAGETWO-56742 and linked it to this one.

@andimov andimov added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Aug 10, 2016
@andimov andimov removed their assignment Sep 2, 2016
@slavvka
Copy link
Member

slavvka commented Sep 19, 2016

The fix has been fixed in 2.1 and will be released as a part of future path releases

@slavvka slavvka closed this as completed Sep 19, 2016
@Ctucker9233
Copy link

@slavvka If we need this fix now, where could we find it?

@slavvka
Copy link
Member

slavvka commented Sep 27, 2016

@Ctucker9233, try to use the patch:

Index: app/code/Magento/Catalog/Block/Adminhtml/Product/Grid.php
===================================================================
diff -u -N -r2d368d0134597257ef5697a4bcbb7f69f2aebed8 -rb80bd55e96f26b745f878169b7ab20af1a0a2a8c
--- app/code/Magento/Catalog/Block/Adminhtml/Product/Grid.php   (.../Grid.php)  (revision 2d368d0134597257ef5697a4bcbb7f69f2aebed8)
+++ app/code/Magento/Catalog/Block/Adminhtml/Product/Grid.php   (.../Grid.php)  (revision b80bd55e96f26b745f878169b7ab20af1a0a2a8c)
@@ -114,8 +114,6 @@
      */
     protected function _prepareCollection()
     {
-        parent::_prepareCollection();
-
         $store = $this->_getStore();
         $collection = $this->_productFactory->create()->getCollection()->addAttributeToSelect(
             'sku'
@@ -184,6 +182,9 @@
         $this->setCollection($collection);

         $this->getCollection()->addWebsiteNamesToResult();
+
+        parent::_prepareCollection();
+
         return $this;
     }

@Ctucker9233
Copy link

@slavvka Forgive my noob question, but how do I use this patch? I am not using composer to update off of the github repo. Can I simply manually add the changes to the files?

@slavvka
Copy link
Member

slavvka commented Oct 12, 2016

@Ctucker9233 yes you can

okorshenko pushed a commit that referenced this issue Dec 14, 2016
[South] Bug fixes backport to 2.1.3
- MAGETWO-56936 [Backport] [Github] Allowed countries for customer address in admin using storeview configuration #2946 - for 2.1
- MAGETWO-57187 [Backport] Captcha JS Error on CMS pages with layout is set to empty - 2.1
- MAGETWO-56742 [GitHub] Sorting & Filters not functioning on New Review page. #5391
magento-cicd2 pushed a commit that referenced this issue Apr 28, 2020
Fixed Issues:
- MC-30925: Customer group API update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

5 participants