Hi,

First, I really appreciate the work done on this module and the Bookings API and I'm eager for the node-version of the project...

The dates that appear when modifying public bookings (at /publicbookings/request/x/update) should be those that were introduced the first time, and a confirmation message should be sent to the user.

Vincent

CommentFileSizeAuthor
#5 publicbookings.497198_2.patch1.39 KBAnonymous (not verified)
#2 publicbookings.497198.patch1.28 KBAnonymous (not verified)

Comments

Anonymous’s picture

Assigned: Unassigned »
Category: feature » bug

Thanks for the bug-report (if the wrong dates are displayed, this qualifies as a bug), I'll have a look after it.

Anonymous’s picture

Status: Active » Needs review
StatusFileSize
new1.28 KB

Could you test the attached patch?

Apologies for the delay.

Anonymous’s picture

Committed this, as another long-standing bug is resolved with it (and I simply forgot to commit it previously). Feel free to reopen, if the above problem persists after either applying the patch or updating to the latest dev (as soon as it is released).
Regarding the email confirmation: With the latest dev of Bookings API a Rules-integration has been added. This would make it possible to contact the client of a booking whenever the booking is changed.

Anonymous’s picture

Component: User interface » Code
Status: Needs review » Fixed
Anonymous’s picture

StatusFileSize
new1.39 KB

Apologies for the quadruple-post ... There was a severe bug in the way PASSPHRASES where generated and clients handled internally, which lead to the missing "update notifications" after a client changed his own booking.
The attached patch (already commited) should fix this (use together with the patch above)

vmal’s picture

Thanks for the modification.

Nevertheless, you have deleted an opening brace at line 117 of publicbookings.public.inc (with the second patch).
With that corrected, it's working fine.

vmal’s picture

Status: Fixed » Active
Anonymous’s picture

Status: Active » Fixed

Thanks - and apologies for the mistake .. I really shouldnt diff under pressure.

Committed again.

The patch to fix the missing brace:

Index: publicbookings.public.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/publicbookings/publicbookings.public.inc,v
retrieving revision 1.7.2.4
diff -r1.7.2.4 publicbookings.public.inc
117c117
<
---
> }

Anonymous’s picture

Status: Fixed » Needs review

And I shouldnt post when tired..

Anonymous’s picture

Status: Needs review » Fixed

Setting this to fixed - feel free to reopen if problems occur with this patch.

Status: Fixed » Closed (fixed)

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