I have webform installed on my site, as well as ubercart.

Webforms has a link /node/id/edit/stock

Ubercart has some stock links of the form admin/store/settings/stock and store/reports/stock

I was editing my webforms, and clicked /node/id/edit/stock on the webform UIand got this Ubercart related error

warning: Invalid argument supplied for foreach() in C:\apache2triad\htdocs\shop\sites\all\modules\ubercart\uc_stock\uc_stock.module on line 136.

Could someone please explain this

Thank you

CommentFileSizeAuthor
#3 task_namespace.patch1.53 KBcha0s

Comments

Island Usurper’s picture

uc_stock also provides a link at /node/%/edit/stock, and your webforms are going to that page instead. Some part of the node is expected to be an array, but it isn't, so it throws an error.

I'm not sure what the best way to solve this is, so I'll leave this issue open.

rszrama’s picture

Version: 5.x-1.3-rc1 » 6.x-2.0-beta1
Status: Active » Postponed (maintainer needs more info)

I'm not sure why webform is using a /stock tab, but I say let's just change it from /node/%/edit/stock to /node/%/edit/uc-stock. Perhaps we should review adding uc- for our other local tasks?

cha0s’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new1.53 KB

Got a simple patch up for the stock task since it seems to be breaking stuff, but other local tasks will be namespaced as well. I post this patch here just so droopie can get the site working. =)

Thanks for reporting.

droople’s picture

I am sure the patch will be helpful to many.

My setup was on a test environment

cheers

rszrama’s picture

Status: Needs review » Fixed

Easy enough. I'll commit this and we can start a new issue that references this one when we have a patch that addresses other local tasks.

cha0s’s picture

For reference, issue has been created at http://drupal.org/node/351389

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.