Bug in Event waiting list

Hey guys,

Hope you had a good TET.

I have an annoying Bug in the waiting list stuff. The [EVENT_LINK] that the website sends out does NOT work when I cancel peoples subscription on the backend (user cancellation over front-end seems to work).

Here is what I received when I canceled someones event attendance (Link does not work): http://munich.international-friends.netindex.php?option=com_eventbooking&view=event&id=216&catid=18&Itemid=1699

Here is what the link for front-end is: munich.international-friends.net/events/...ake-tax-declarations

Found the bug.... It misses out one / after the .net and the index.php. How do I solve this?

Hi Tim

Still having the holiday now but I am now having more time to work than in the last 4 days. Could you please let me know step by step what I can do t see the error?

I will then check and solve it

Tuan

OK that one is easy...
(the problem is a missing / )

Here is for reproducing it:

_Create event and subscribe a couple of people to it. Including a couple in waiting list (ie the event is full).
_Go to back-end registrant management.
_Set one of the subscribed people FROM THE BACK-END to cancelled.
_It will send out the waiting list opportunity email to the people on the waiting list. The link in the email will be missing the / between the domain and the index.php.

Could you tell me which php file (and best, how) I should change? It is a simple spelling error as far as I can see.

Hi Tim

Try to look at the file components/com_eventbooking/helper/helper.php, method notifyWaitingList, you should see the code and can change it to meet your need. Then please report back what you changed, that would help saving me sometime

Regards,

Tuan

Euh... I am a biologist... No idea about php...

Just so that you can see the problem clearly. Look at this link taken out of a waiting list notification (yes it happens in all of them, this is just an example): http://munich.international-friends.netindex.php?option=com_eventbooking&view=event&id=216&catid=18&Itemid=1699

Locate the index.php in the link. There is a / missing in front of index.php.

The working link would be munich.international-friends.net/index.p...catid=18&Itemid=1699

Looking at the notifyWaitingList method Tuan mentioned, it uses JUri::getInstance()->toString to form the url.

My site was producing URLs with the same missing '/' while using the exact same JUri::getInstance()->toString command... because of a plugin conflict.

It would be a surprise if the problem was caused by the exact same broken third-party plugin... but stranger things have happened. @Tim, double check your plugins, and see if you have a MVC Override system plugin running, and disable it. It might shut down some custom code overrides make to otherwise un-overridable modules/components/etc. (check your template/YourTemplate/code folder (if it exists) to see if there is anything crucial being shutdown).

@James: Nope havent got that plugin. Also havent got the template/YourTemplate/code folder

@Tim: Well, at least you know that it isn't that plugin causing an issue Only if you had that plugin would you have a /code folder.

Tuan should investigate further, but I think you should be able to get *your* site providing proper links by making the following change in components/com_eventbooking/helper/helper.php (warning -- I have not tested this modification, but I think it should do the trick for you. Make a copy of the original file so that. if it doesn't work, you can revert back).

Look for the two occurrences of JUri::getInstance()->toString(array('scheme', 'user', 'pass', 'host')) (lines 2488 and 2492 in my install).
Immediately after the [edit: SECOND occurrence of ] JUri::getInstance()->toString(array('scheme', 'user', 'pass', 'host')) and before the existing . (period), add
In the end, you should have something that looks like: [edit Apply only after the SECOND (the else) occurrence ]

Tuan will be able to confirm whether this is a bug that needs fixing the core package, or if it is just something that is happening on your site.