Deposit Type option not working

Hi,
Sorry, but this is not obvious.
The wording of the ‘cost’ line is set system wide in the settings tab

The setting in the event just impacts how any checkout is calculated.

If you have a mixture then there is no option at the moment except to use some generic text.

Thanks for your swift reponse Alan. So, if I understand correctly and for the benefit of anyone else looking at this, the deposit type radio button options for each event have no effect at this time.

I have tried changing the text options in ‘settings’ but if I change and save ‘per person’ to ‘per event’ the change isn’t saved and reverts to ‘per person’.

Is there another way to change this text? code maybe?

• This reply was modified 1 year, 10 months ago by pauljldev.

It will have no effect if you are not taking payment. I agree it is a silly design error.

That said, the plugin has been around 9 years plus and I took over support and maintenance about 2 years ago and so so far you are the only user that has noticed or cared enough to point this out, for which I thank you.

I have added a task to make this better to the development list.

That’s not quite true because if you wish to show the cost of the event the cost/desposit line will show in the event details and will of course show the incorrect information.

I think the only work around for this particular site seems to be not to display the cost line and include cost/deposit details in the description of the event. I support other sites with this plugin but all of their events would be per person so obviously not an issue.

Would be great if this could be addressed in one of your future releases. It is a great plugin none the less. Thanks.

i missed

per person’ to ‘per event’ the change isn’t saved and reverts to ‘per person’.

that is a separate issue that needs fixing

Hi Alan,

Thanks for this.

Although I don’t like doing this I have made a small change to the qem-event-build-functions.php code to correct this behaviour. I judged that this was an isolated change that had no impact on any other part of the plugin and I know that it will get overwritten with an update but it’s such a small change I think I can live with that (and at the end of the day it will just revert to current behaviour).

The perevent/perperson constants get saved/retrieved with each event and it was just a question of using the custom object as opposed to the event object and replacing the constant with display text.

I’m happy to share the snippet if this would be of help, just let me know.

Of course I most welcome contribution to the free code. You can also use https://github.com/alanef/quick-event-manager and create a pull request if you like

I’ve tried to do a pull request (this is new to me), not sure if it worked OK but let me know if not and I’ll try again.

Not quite a pull completed – but I can see your change
https://github.com/alanef/quick-event-manager/compare/master…pauljldev:quick-event-manager:master

I’ll create something a bit more flexible for the next release to allow setting the texts in the option – I’ll let you know before release so you are ready to set the new options.