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.On a page 'admin/store/customers/orders/%uid' there is a link "Create an order for this customer" that points to 'admin/store/orders/create/%uid'. But the latter page acts like a regular order creation page for customer number 0, i.e. %uid is just contained in the path but not used in the code.
As far as I can see, there is no path like 'admin/store/orders/create/%uid' in hook_menu() (only 'admin/store/orders/create'), functions uc_order_create() and uc_order_create_form() don't use %uid either.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 1247090-create-order-take-3.patch | 804 bytes | muhleder |
| #4 | 1247090-create-order-take-2.patch | 2.85 KB | TR |
| #2 | 1247090-create-order.patch | 2.79 KB | TR |
Comments
Comment #1
ergophobe CreditAttribution: ergophobe commentedSame problem here on UC 2.4 so it's not a new problem.
Comment #2
TR CreditAttribution: TR commentedPlease try this patch and see if it works for you.
Comment #4
TR CreditAttribution: TR commentedThat was a real test failure. Let's try this again.
Comment #5
TR CreditAttribution: TR commentedCommitted.
Comment #6
TR CreditAttribution: TR commentedNeed to examine whether we should do the same thing in 7.x-3.x where the create order page is a bit different.
Comment #7
longwaveWe need to take a slightly different approach in D7, but fixed all the same: http://drupalcode.org/project/ubercart.git/commitdiff/1218a93
Comment #9
muhleder CreditAttribution: muhleder commented#4 Doesn't quite work from what I can tell
$account is assumed to be loaded, but it's actually $uid we get from the url.
This patch should fix.
Comment #11
muhleder CreditAttribution: muhleder commentedActually, scratch that, it does work already. I probably just needed to clear caches after applying the patch.