Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Allows to re-order previously ordered products directly from the orders list. Reordered line items will be cloned to the Cart.
This project is similar to the Commerce Reorder, but allows to reorder an individual line item, not the entire order.
Views
Provides "Reorder" button field handler.
Rules
The Rules actions can be applied using "The line item is reordered" condition or "After the line item has been reordered" event.
Project page
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/artreaktor/2501091.git commerce_reorder_lineitem
cd commerce_reorder_lineitem
Comment | File | Size | Author |
---|---|---|---|
Commerce_reorder_lineitem.jpg | 40.74 KB | artreaktor |
Comments
Comment #1
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxartreaktor2501091git
We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #2
artreaktor CreditAttribution: artreaktor as a volunteer and commentedComment #3
dillix CreditAttribution: dillix commentedHi, artreaktor
You should set git default branch as described here: https://www.drupal.org/node/1659588
Comment #4
artreaktor CreditAttribution: artreaktor as a volunteer and commenteddillix, thanks!
Comment #5
dillix CreditAttribution: dillix commentedHi, artreaktor
There are some issues with your code: http://pareview.sh/pareview/httpgitdrupalorgsandboxartreaktor2501091git
You should fix Drupal code standards.
Also you didn't fix default branch issue.
Comment #6
artreaktor CreditAttribution: artreaktor as a volunteer and commentedDilix,
following your advice I've selected the default branch just before #4 was written. Rechecked just now: the default branch IS selected.
As to #5
As you may know the Views class naming convention doesn't meet the PHP standards, so I don't have a clue how to overcome this.
Thanks
Comment #7
artreaktor CreditAttribution: artreaktor as a volunteer and commentedComment #8
EvanSchisler CreditAttribution: EvanSchisler at Acro Commerce commentedManual Review:
Automated Review
[Best practice issues identified by pareview.sh / drupalcs / coder. Please don't copy/paste all of the results unless they are short. If there are a lot, then post a link to the automated review and mention that problems should be addressed.]
Note that perfect adherence to Drupal Coding Standard is NOT a reason to block an application, except for total disregard of them. However, modules should follow them as closely as possible.
Manual Review
You are missing an introduction, the configuration heading just tells us what your module does and not how to configure it, you are missing the requirements header(very important since your module does have dependencies.) Please check out the template for README.txt files.
If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.
This review uses the Project Application Review Template.
Comment #9
EvanSchisler CreditAttribution: EvanSchisler at Acro Commerce commentedComment #10
klausiThe readme style alone is surely not an application blocker, anything else that you found or should this be RTBC instead?
Comment #11
EvanSchisler CreditAttribution: EvanSchisler at Acro Commerce commentedAs long as the readme style and the list of pareview.sh errors listed by dillix are not blockers, then yes.
Comment #12
PA robot CreditAttribution: PA robot commentedFixed the git clone URL in the issue summary for non-maintainer users.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #13
artreaktor CreditAttribution: artreaktor as a volunteer and commentedEvanSchisler, klausi, thanks for the review!
I've improved the README.txt file to meet the recommended template.
As to pareview.sh results I've already fixed all the issues I was able to fix.
Comment #14
artreaktor CreditAttribution: artreaktor as a volunteer and commentedComment #15
artreaktor CreditAttribution: artreaktor as a volunteer and commentedComment #16
spacetaxi CreditAttribution: spacetaxi commentedManual Review
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.
If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.
This review uses the Project Application Review Template.
Comment #17
spacetaxi CreditAttribution: spacetaxi commentedComment #18
PA robot CreditAttribution: PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.