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.
As for #2126485: Can we have a 2.x-dev release? "commerce_coupon_review_pane" is still referenced in checkout but never created, we should review all the checkout process views to see if there are more cases like this.
Comment | File | Size | Author |
---|---|---|---|
#5 | 2143113-5.patch | 3.1 KB | roderik |
Comments
Comment #1
torgosPizzaI believe I found the issue, and it's twofold:
1) I had loaded the new module code on top of the old, meaning there were still some leftover crufty Views defaults there.
2) The variable "commerce_coupon_review_pane_view" needs to be deleted from the database, because the old Coupon module had set this variable and the new 2.x still does reference it. This means an upgrade path will need to simply remove this variable and remove/disable the old pane view.
Everything else worked fine, though, after uninstalling 1.x and re-installing a clean 2.x.
Comment #2
pcambraThanks for the feedback @torgosPizza!
Checking this further I see that the checkout review pane view used is now called "order_coupon_list", I guess we should try to keep the name from 1.x to 2.x as there's no need to change it, I think the name was "commerce_coupon_review_pane"
Comment #3
roderikFor completeness: I installed commerce_coupon 2.x 'new' (i.e. after completely uninstalling 1.x), and everything works fine. So it's an update issue.
The view name is not directly referenced, but through a variable. So that variable value should be updated. I guess it should only be updated if the "commerce_coupon_review_pane" is actually not present anymore, so that leads to the uploaded patch.
Note #1: I spotted that the update functions are incorrectly numbered (now that this is actually the 2.x branch). So included that in the same patch.
Note/question #2: Why are we actually renaming the view from 'commerce_coupon_review_pane' to 'order_coupon_list'?
If not, this new update isn't necessary. But obviously I only ask myself this question after coding it... :)
Comment #4
pcambraThat's the point :), we shouldn't, IMHO the patch should be creating the view as 'commerce_coupon_review_pane' instead of 'order_coupon_list'
Comment #5
roderikMeh, crosspost. I'm too slow for this; was coding/testing a patch before commenting, but missed your conversation in the meantime :)
Here's a quick reroll that works in the site I'm tinkering with (after I manually changed the variable settings back). The .install patch that I picked up along the way, is split into a separate file.
Comment #6
roderikFailed to change the first comment line in patch 5a.
Comment #7
pcambraThanks for the patch, -5 is the one right for this issue
For exploring upgrade paths, we've got https://drupal.org/node/2125383 better
Comment #8
dpolant CreditAttribution: dpolant commentedThis is pretty old and I am reasonably sure the problem has been fixed. I am going to close for now.