On the Module page, for Reservations Pricing submodule it says:

"Adds additional option to Reservation setting form to include a Price Set"

But on the Reservation setting form I'm not seeing this option

Also, when I create a Content Type for a Reservation item

Then click on Edit that content type, it returns this error

Fatal error: Call to undefined function reservations_pricing_get_pricing_tids() in /var/www/sites/cm_starterkit_easy/sites/all/modules/reservations/reservations.module on line 1213

When I disable the Reservations Pricing module this error goes away

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pimok3000’s picture

i get exactly this as well with the latest DEV version from 25th of may 2013

Fatal error: Call to undefined function reservations_pricing_get_pricing_tids() in my_mutliinstall_drupalpath/all/modules/reservations/reservations.module on line 1219

deactivating the reservations_pricing modules makesthe error go away and admin/reservations/manage/current_inventory work again

caspervoogt’s picture

Function reservations_pricing_get_pricing_tids() is not defined anywhere in 7.x-2.x-dev or its submodules. Maybe it existed in an earlier version and was lost. Looking for it.. Will report back here.

caspervoogt’s picture

I checked the different branches of the project and did not see the function anywhere. It seems to be intended to detect pricing-related terms, but the documentation doesn't mention creating taxonomy for this, and the code suggests that somehow the resource item content type should reference the pricing vocabulary, but the content type doesn't offer such an option in the Reservations Settings tab.

I have wrapped the function call in an:
if(function_exists('reservations_pricing_get_pricing_tids'){

Patch is attached. This doesn't solve the potential underlying problem of the missing function ... just hides the fatal error. Not sure what that function is needed for or how it's meant to be implemented. I have not yet fully tested the pricing submodule either, so it may not be working regardless of this change, but at least this way you can have the pricing submodule enabled without fatal errors.

caspervoogt’s picture

forgot to attach the patch! Here it is.

p.s. I realized afterwards that the patch has tabs (rather than double spaces).. let me know if you need me to re-roll it.

Djundi’s picture

Issue summary: View changes

Sorry guys, were i must paste this patch, i do not find the path, how to fix this, i em new in Drupal :)

IT Munger’s picture

Applying a patch manually: https://www.drupal.org/node/534548

kenorb’s picture

Status: Active » Needs review
kenorb’s picture

Assigned: kreynen » Unassigned