Forum Replies Created
-
Posts
-
Because that SQL query is heavy, you have to select the fields you want to be shown in the multi-select dropdown just left of the button “Filter persons”.
Concerning the PDF: what placeholder do you use to show that field? Your template should contain something like #_FIELDVALUE{mobil}Indeed, fixed that part now (the html was stripped out, and \n or \r doesn’t show in html output …).
Your second attempt for the use of filters is the correct one (wordpress method). Don’t forget to put this in functions.php in a child theme, so when the theme updates your changes aren’t removed.
This should do the line break:
$my_separator=”<br \> “;Latest dev should fix the html output.
Also, from the readme:
* Added filters eme_event_categories_sep_filter, eme_event_categorydescriptions_sep_filter, eme_linked_event_categories_sep_filter
These control the separator for #_EVENTCATEGORIES (default: “, “), #_EVENTCATEGORYDESCRIPTIONS (default: “, “) and #_LINKEDEVENTCATEGORIES (default: “, “)Concerning the event description: if you really mean the description (and not the single event format), check the option “Enable placeholders in event notes”
That’s currently not possible unless you first apply this change (untested, so feel free to test …): https://plugins.trac.wordpress.org/changeset/1792552/
For order, you can then use the id-option (that allows multiple id’s), or multiple category id’s too. The separator between categories is something you need to figure out on your own (you can use conditional tags if you want, or play with css …)Try this change for cat order:
https://plugins.trac.wordpress.org/changeset/1792545/For the first feature request, see:
https://plugins.trac.wordpress.org/changeset/1792023/Sorting by exclude is impossible btw … sorting by include is possible, but not now 🙂
Doing that would cause me to build a whole lot of extra logic into discounts, because: what to do if 4 places are already used, and a person reserves 2 more? Or more: what if several registrations are ongoing and those together surpass the limit? And more: what if those already saw the actual discounted price via #_DYNAMICPRICE and at the moment they press the button to actually book, they see that the price changed? Only one way then: to refuse the booking …
While I see your point, currently you’re the first one to ask for this. And I think that’s exactly why the program-type discount was created, so you can achieve what you want by coding that part yourself …
Or maybe: what if you just show the discount field as long as less than 5 places are reserved? You can use the conditional tag [eme_if] for that, see
http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-1-conditional-tags/Fixed per seat means that a fixed amount is being deducted from the price per seat, but it still only uses 1 discount. The discount usage increases per discount code used.
Sorry, but I’ve been busy with other bugs, daily work and end-of-year preparations …
Good you got it to work. The action hook eme_insert_member_action is in the latest version now and also documented.I specified my example based on your query: “If I could specify number of seats for each price that would do it, I would just make five seats at 310 individual registration, but that does not appear possible. Seems that number of seats can only be set for the event as a whole, not for the different prices.”
Multiseat/multiprice events are exactly that. But it is a seat category, meaning a booking in one price category will not decrease the total number of seats in another.
I’m guessing you need a discount of type ‘code’, where you then code your wanted result.
See http://www.e-dynamics.be/wordpress/category/documentation/17-discounts/ for a starting example. You’ll need to do some coding yourself there …https://en.wikipedia.org/wiki/RSVP_(invitations)
In other words: to be able to make reservations in EME
Which EME and which WP version? Because line 326 in version 2.0.13 of EME (the latest released version) contains this:
if ($payment['member_id']>0) {and that can’t trigger a syntax error …
If you decide to no longer use the plugin, please share as to why so I might improve it based on that feedback.
See the EME settings, there are options you can click that will remove the tables upon deactivation.
That’s outside the scope of EME, but you need to start by examining $_POST when entering a member. I’ll add an action hook for new members in the next version, so you can work based on that info.
Check the doc on conditional tags (with your question as an example):
http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-1-conditional-tags/EME doesn’t really have that feature, but using conditional tags it is easy to achieve. See:
http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-1-conditional-tags/Use something like this in your event list format:
[eme_if tag='#_DAYS_TILL_START' lt='7'] Show my event #_EVENTNAME [/eme_if]Ok, I think I got it. Although I don’t play with that header, apparently (in the past), this was implemented:
add_filter('locale','eme_redefine_locale',10);
and the said function:function eme_redefine_locale($locale) { if (function_exists('pll_current_language') && function_exists('pll_languages_list')) { $languages=pll_languages_list(); if (!$languages) return $locale; $locale=""; foreach ($languages as $tmp_lang) { if (preg_match("/^$tmp_lang\/|\/$tmp_lang\//",$_SERVER['REQUEST_URI'])) $locale=$tmp_lang.'_'.strtoupper($tmp_lang); } if (empty($locale)) $locale=pll_current_language('locale'); } return $locale; }I think it had to do with polylang stuff in the past (but don’t ask me why again, it was version 1.5.18 or so (3 years ago …).
But even version 1.7.x had this code (and all newer), so it didn’t really change the last years. Maybe polylang changed stuff (as they are entitled to do of course).
So currently: remove line 181 in events-manager.php (saying:
add_filter('locale','eme_redefine_locale',10);
).
Check if that fixes it. Of course EME still needs to detect somehow the lang by polylang, so it should be the setting “The language is set from content” apparently (again: something from years ago …). EME is not fully polylang compliant, although using EME templates you can go far already …Then maybe your theme is doing something with the language_attributes filter. Like I said: I don’t influence that header in any way. Maybe you can check your theme support, or check out if any other plugin might cause this. And check your wordpress language settings. If your theme requires something specific, I’ll gladly take a look at that requirement but I can’t support any pro-themes myself.
Sorry, in the backend the header is generated by the function _wp_admin_html_begin, which also calls language_attributes().
I have no influence on that header whatsoever, and in my theme the lang=”en-US” appears just fine with EME loaded (check out this forum-site).
Your theme should call language_attributes() in it’s header.php file (see twentyseventeen as an example), and that function definition can be found in wp-includes/general-template.php.
The only reason for the language part not to be present would be your blog having no language set … (which is impossible I think)I don’t actually store the used voucher with the booking (but only the calculated discount), so that’s currently not possible.
I was a bit too quick yesterday, this fixes it completely now:
https://plugins.trac.wordpress.org/changeset/1787515/Also, for the discount calculated, you can use #_TOTALDISCOUNT in your mails.
argh … I tested the code with payments and paypal and applied discounts, but not the dynamic pricing placeholder … let me see
I just figured it out while playing on your site …
This should fix it:
https://plugins.trac.wordpress.org/changeset/1787329/If something doesn’t load, check your browser console for hints. A javascript error of some kind might explain this.
Is this reproducible? Meaning: always for 100% discount code?
There’s currently no discount placeholder for mails (yet), but it is a good suggestion.You have the creation and modification timestamps in the db, and you can get those via a placeholder.
Example: #_EVENTFIELD{creation_date}
Others creation_date_gmt, modif_date, modif_date_gmt (see function eme_create_events_table in events-manager.php).Edit: currently not foreseen to get this info in the event listing (in EME backend), but it is very easy to add …
You create a category and note down the ID.
Then on a regular WP page, you enter the [eme_calendar] shortcode with the category option, see
http://www.e-dynamics.be/wordpress/category/documentation/6-shortcodes/eme_calendar/For payments: did you enter your payment gateway details? See EME options, section Payments
If that is the case, it means your theme is doing this. Luckily eme allows to set/change any styling. See http://www.e-dynamics.be/wordpress/category/documentation/9-changing-the-layout-css/
Was a bug, fixed today. See
https://plugins.trac.wordpress.org/changeset/1785596/What is this “it recognized that the database exists and I would lose my events if I reinstalled”? EME doesn’t recognize that the database exists (well, it does, but it will not tell you and silently use what is there) and you never loose the data you have.
EME has already several only payment gateways integrated. But why would you still need another member plugin? I’m guessing writing a simple function that denies access to a page if not having a specific membership isn’t that difficult …
Use correct double quotes (not word-like ones) and it will import just fine (just tried your lines here).
[eme_events] has an option to only show specific categories. So I’d just create a page per category and then your button can link to that page.
Well, indeed if the price is 0 after discount and auto-approve is selected, it should be auto-approved. I’ll take a look at that.
(I’ve split this into a new thread, please don’t hijack threads …).
EME only goes to paypal if the price is >0. So this means the price is indeed not 0 when that decision is being made. See line 402,403 in eme_rsvp.php. The function call there (eme_get_payment_price) takes the discount for each booking into account too.
So: how do you know the price is 0 after booking but before going to paypal? And/or a demo would help too …If you just want to show a single specific event, use the shortcode [eme_event], see http://www.e-dynamics.be/wordpress/category/documentation/6-shortcodes/eme_event/ (templating and all).
Inside your single event format. Don’t use the eme_add_booking_form shortcode if you don’t have to, use #_ADDBOOKINGFORM inside your single event format too.
[eme_if tag='#ESC_RESERVEDSPACES' ge='#ESC_TOTALSPACES']<span style=”color:#F29849;font-weight:bold;font-size: 110%;”>This event is sold out!</span>[/eme_if] [eme_if tag='#ESC_RESERVEDSPACES' lt='#ESC_TOTALSPACES']#_ADDBOOKINGFORM[/eme_if]I just took a look at your site, and your form is present on the page, but completely hidden (just do a view-source).
The reason is that your form is not valid: #_PRICE1 is no longer supported, it should be #_PRICE{1}, and #_SEATS1 should be #_SEATS{1} (since 2014 already: http://www.e-dynamics.be/wordpress/legacy-placeholders/ )That is indeed the place where such an action could go. I’d do it like this:
foreach ($ids_arr as $member_id) { $res = eme_member_set_paid($member_id); if ($res) { if ($send_mail) { $member = eme_get_member($member_id); $res2 = eme_email_member_booking($member,$action,$queue); if (!$res2) $mails_ok=0; } if (has_action('eme_member_paid_action')) do_action('eme_member_paid_action',$member_id); } else { $action_ok=0; } }Then, in your action, you can call eme_get_member to get the complete memberinfo for that memberid (and eme_get_answers for custom answers). There’s no field yet for passwords though, although adding a new fieldtype is very easy to do (adding it in the function eme_get_fieldtypes should be sufficient).
Make sure to link the newly created WP user to the person in EME (the person is linked to the memberships for that person) using the column wp_id in wp_eme_people. And for passwords in WP, use wp_set_password or wp_hash_password.See this thread for a starting example:
http://www.e-dynamics.be/wordpress/forums/topic/put-an-availability-traffic-light-image-to-the-event-single-page/Then why not just add that info to your form markup …
Also, you can’t select more than the allowed max number of seats, how do you get this message? And what is the exact message (because that text doesn’t exist)?Check out the conditional tags (eme_if): compare the number of booked seats with the total and show a message if full (also search this forum for examples)
Are your events RSVP enabled? What is the format of your single event? Does it include the placeholder to show the RSVP form? What is the format of your RSVP form itself?
No, thank you for insisting on this problem! Sometimes I just don’t see the bug when there really is one!
So, the template with id 37 doesn’t contain a newline after the eme_if-endtag.
Then the error is in the function eme_nl2br_save_html … and I think I found the reason.
It is in the function eme_nl2br_save_html, where I loop through all the lines if html is found (and in your case: the strong-thing is html, so I’m guessing it should be ok if you didn’t use html). At the end of each line, I add a br+\n, without checking first if I should really be adding that at the end …
I now implemented a check for that, so could you try if this changeset fixes it for you:Problem found (imho: in there for quite some versions …)
Fixed here:
https://plugins.trac.wordpress.org/changeset/1781617/Ok, I’ll check later this evening myself with a test-user. This might be a permission-check problem (or not …)
Ok, this is more info (but please in the future use the “code” button).
Your header/footer template are ul, but your entry template doesn’t use “li”. This is not valid html …
First change either your header/footer templates to something else, or if you want to keep the ul, change your template 3 to:
[eme_if tag="#ESCY" value="2017"]<li>#_STARTDATE — #_LINKEDNAME — #_STARTTIME to #_ENDTIME</li>[/eme_if]This is not possible, the attributes are used for selection info and can’t be consulted on their own (just like any other custom field can’t be listed)
You still haven’t shown the format you use. I’m guessing there’s a break/newline before and after the [eme_if] section.
And in fact you should blame wordpress for all this: their editor removes br-lines when switching between visual and text editor, and since I don’t know in which editor you made this, I do the following (simple logic, the same is used by wp): replace all leftover br-lines by newlines, and then replace all newlines by br (in case of html of course).
So a newline before and after your [eme_if] will cause this, but as long as you don’t show the format, I can’t tell you for sure.The next version will support:
color
date
datetime-local
email
month
number
range
search
tel
time
url
weekLike I said: “it should all go through the wordpress translation site”. If it is a language already 100% translated in WP, then local files are ignored. Check:
https://translate.wordpress.org/projects/wp-plugins/events-made-easyI’m guessing you need to read this 🙂
http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-11-categories/The url-placeholder is what you’re looking for.
Depends on the language. And you need to update the mo file too. And also: I no longer update the local .po file, it should all go through the wordpress translation site (see wordpress for that).
You have a permalink for each category (the url to your events page + ‘/cat/
, with being the sanitized cat name). Apparently I don’t show this link yet, so this change should fix it (and allow to change part of the permalink too):
http://plugins.trac.wordpress.org/changeset/1778773What does your rsvp form look like? The check-value for dynamic data (your #_SEATS) needs to be present in your form (and not e.g. #_SPACES, #_SEATS{1}, …).
Sorry, I meant “eme_eventAction=paypal_notification”. So paypal is connecting back to you (that’s good), but your server posted a 503 reply to it (“Service unavailable”). So you should try to find out why your webserver refused this (mabe some security plugin?).
Check for eme_eventAction=paypal_ipn in your webserver logfiles. If that doesn’t appear in there, it means paypal is not connecting back to you. I’m providing the ipn-url for them, but that can be overridden in your paypal settings, so you need to check there too (and maybe easier: ask paypal for a little help, they’re more than willing to help you).
Concerning the IPN mail template: it is just like any other booking mail template, the same placeholders apply. It is empty by default so if you want to receive an email upon IPN from a payment gateway, fill out the subject and content.First: your usage of eme_if is wrong, see the doc: http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-1-conditional-tags
Second, in the same doc, check out #_IS_ALLDAYIf the divi template is free, I could take a look. But I guess for now you should ask the divi-coders for support on where their counterpart for the search is stored.
Adding multiple people as attendees at once is not possible (you can ask for more info concerning a booking using custom fields and/or dynamic fields though).
So it is as I suggested. Just remove the empty line before and/or after the [eme_if] (so it “sticks” to the previous/next line) and you’re good to go.
Make sure to check the ‘text’-view. Also, show the complete template you use. The eme_if logic won’t show the text if the evaluation is false, meaning (in your case) it will only show lines from 2017.
Don’t forget: an empty line at the end after your ‘if’ might be the cause too … (that gets translated to br-tags). Check the resulting html to be sure of that (add some text immediately after the closing eme_if to test that)Creating custom buttons with custom links is outside of EME functionality.
Either google on how to create your button, or use some wordpress plugin (there exist some that help you in creating buttons, e.g. maxbuttons).EME doesn’t add blank lines, so it is probably in your template. Based on your description, I’m guessing you have br-tags in your template but outside the eme_if logic, meaning they will show for every booking.
No need for DB queries, just use #_AVAILABLESPACES (see ttp://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-2-events/ )
Together with the use of conditional tags (http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-1-conditional-tags/ ), you can check the value of this placeholder and decide to show something, see this example (to be placed in in your single event format):[eme_if tag='#_AVAILABLESPACES' lt='10'] [eme_if2 tag='#_AVAILABLESPACES' gt='5'] <p>Less than 10 spaces but more than 5 spaces remaining</p> [/eme_if2] [/eme_if] [eme_if tag='#_AVAILABLESPACES' lt='5'] <p>Less than than 5 spaces remaining</p> [/eme_if]In fact: you’re totally correct! There is a bug with attributes of type “textarea” (the default type).
I released 1.0.27 (and updated the doc too). So if you want just a text-input field, do this:Thanks, I moved this to “Tips”!
Btw: the same function can be used for locations.
And also: check out eme_is_single_event_page(), eme_is_single_location_page() and their companions (in eme_functions.php)Currently the “send mail” mass action is not possible for participants of past events, I’ll add that option later on.
EME has a lot of filters and hooks that you can use, it all depends on the use case. Combined with conditional placeholders , you should be able to do what you want, but it’ll require some coding though.
Changing the title of the page is currently not possible. I imagine a filter would help, but an easy wor around would be to use a page template without title and just add your own via the eme single format template
A little bit more explanation is needed: is the attribute visible in the form? Which form did you change? And where in the event did you add the attribute then?
Ok, that is not a multibooking event, but just a normal multiprice event (your event does have multiple prices, no?).
Check out: http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-4-rsvp-mails/
You need to use #_RESPSPACES{1}, not #_RESPSEATS{1}@dcorfan: it is fixed with this change (will be in the next version):
https://plugins.trac.wordpress.org/changeset/1771668/While EME currently can’t do this on its own, using the [eme_if] conditional tag you can check the number of seats booked in the first group and no longer show the form in the event all seats of group1 are taken.
The notation #_RESPSEATS{1} etc … is for multiprice events, not for multibookings.
Read carefully the mail-section here:
http://www.e-dynamics.be/wordpress/category/documentation/6-shortcodes/eme_add_multibooking_form/@eric r: sorry, indeed you can use #_ATTENDEES too
@dcorfan: it is possible that the the header/footer templates aren’t parsed for placeholders. I’ll check …
Edit: indeed a small logic error prevents the placeholder replacement from the settings (it will work if you use templates). Expect a fix this week (well, today in fact, I hope to release this week).You’re correct, a logic bug crept in while adding dynamic fields …
This should fix it (or download the dev version):
https://plugins.trac.wordpress.org/changeset/1769300/In fact 2 files changed: js/eme.js and eme_formfields.php, so you can download those completely via that link and replace them
Read this remark for the event start/end date:
In case of a recurrent event, use the beginning and end date to just indicate the duration of one event in days. The real start date is determined by the recurrence scheme being used.
EME has no real logic to know the “next” event, but you can use the [eme_events] shortcode and limit to scope=future and only show 1 event. And then, in the template used for that shortcode (via the template_id option), you can use #_EVENTID, which gives you the id of the next event. See the doc on template_id, template_id_header/footer here: http://www.e-dynamics.be/wordpress/category/documentation/6-shortcodes/eme_events/
Use something like this as content in the template (in fact another shortcode is used here):
[eme_bookings id=#_EVENTID]Glad to see someone using the memberships now 🙂
I’ll take a look at the bug, thanks for that.
Concerning the member: you need your membership form configured and set up correctly, otherwise you can’t enter anything for the member when signing up. See http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-13-membershipform/But you’re correct, when it is not set, I should give a warning (or maybe not allow a membership to be created with this part empty).
Ok, I thought of several things:
– first: indeed, the image behavior changed, but only in that way that the image returned by EME is now the one returned exactly as given by wordpress (this was not the case before)
– second: you have huge images there, consider using either some thumbnail format, or use a smaller image (look for #_EVENTIMAGETHUMB in the documentation: http://www.e-dynamics.be/wordpress/category/documentation/7-placeholders/7-2-events/ )
– to get the default image as it used to be returned by eme, use this:
<img src="#_EVENTIMAGEURL">The image returned obeys your CSS rules and all, so if you don’t want it distorted: change the css accordingly, or use #_EVENTIMAGEURL + own html-img tag and custom width/height …
Why not just use a separate rsvp template for that event? That way you can just put #_SEATS somewhere (without label), indicate “attendance” for the event and be done with it 🙂
EME does validation that #_SEATS (or #_SPACES) is present in your template, but before doing eme_if-validations.
Now: EME can also hide this for you: just set the min seats to book to 1 and mark it as an attendance event. Maybe I’ll expand that logic in the future even: if min and max are equal, it can also be hidden …Well, I just checked the code, and the bookings-array should always be there.
Is this a special type of event? Or empty seats booked? Can you check the html via firefox debugger to see if in the received html the following field is defined:
bookings[5][‘bookedSeats’]
?
And/or a public example I can check?Well, an empty htmlmessage is weird … I’ll check
For the rest, the for-loop it skips is indeed the one for multibookings, so that part is ok.When a payment arrives through a payment gateway, it is normally marked as paid automatically.
Also, you can set up mails (IPN notification template) to alert you of these, and those mails can contain all regular booking placeholders (like the normal ID for example).
I can add the payment random id (your “item_id”) to it, but it is a long string and difficult for you to search in.
Is the interface from the payment gateway to your site not working as expected?
The unique nbr: see the placeholder doc for booking mails (concerns Belgium banks).Then read the usage doc:
http://www.e-dynamics.be/wordpress/category/documentation/3-installation-and-usage/Btw, I tested on my site: no issues.
If you see the 2 yellow bars, it means there’s a javascript error on your page preventing things from working. Can you check your browser console?Please answer this: “Which options are you using in it”
For all options in [eme_events], see:
http://www.e-dynamics.be/wordpress/category/documentation/6-shortcodes/eme_events/
(by default only future events are shown, see the ‘scope’ parameter)Ok, now we know what shortcode you’re using. Which options are you using in it and what would you like to see as a result?
Both currently work/act as coded, so yes: it is normal (for now). The mailing can’t be solved, but I’ll take a look at the payment thing (it couldn’t be done before, but the code has evolved and it should be reasonably easy to change the custom payment info for multibookings now).
A bit weird … but i did just change some stuff concerning the forms and messages and such. Maybe you could try the dev version?
The next version of EME will show php generated messages in ajax replies on screen and fail more gracefully.
If you require attendance and set the minimum seats to book at “1” in th rsvp settings, the yes/no will not show.
This also goes for multibookings, but of course there you probably don’t want attendance everywhere. So there the min. should be 0 and you need to do some evaluation yourself using a hook. See http://www.e-dynamics.be/wordpress/category/documentation/12-hooks-and-filters/ , the example “An example for multibooking, to check that at least 1 seat is booked in total” is there exactly for you.This thread talks about the calendar. If you’re using the calendar shortcode: which options are you using?
Can you explain your problem? The mentioned change is already in the code for months (and in the released version). If you have a problem not identical to the one in this thread, please open a new one.
if the wheel keeps spinning (and I assume it works without encryption), then you have a php error somewhere, preventing the rest from working. Check your browser console for hints and also your webserver error logfiles. It might be that the paypal encryption requires some php lib or access to a dir or so …
I’ll check on my side for a possible newer version for the paypal php files.This should help:
http://plugins.trac.wordpress.org/changeset/1762277Thanks for insisting on this problem, and you’re correct: I forgot to add the auto_increment for new installations to the answer_id column (although currently I don’t need it …)
So I’ll add it again, otherwise indeed it will fail to add extra records. I’ll release an update today.It should indeed appear in the answers table.
But 2.0.1 should be ok.
These are the columns you should have in that table:
answer_id int(11) NOT NULL,
booking_id mediumint(9) DEFAULT 0,
person_id mediumint(9) DEFAULT 0,
member_id mediumint(9) DEFAULT 0,
field_id int(11) DEFAULT 0,
answer text NOT NULL,
grouping int(11) DEFAULT 0,
occurence int(11) DEFAULT 0,Check your browser console for javascript errors … probably you don’t have the google maps API key in EME
Read the doc on dynamic fields, #_FIELDS is not appropriate there, use #_DYNAMICDATA
But if you have problems with storing the data, it won’t show anything.
Check your logfiles first, maybe there’s some database problem …EME can do that, but for that it needs to know who you are when you registered, and that is only possible if you’re logged into WP as a user.
In EME, when defining the RSVP parameters, you can check the option “require wp membership”
And then you can use #_ADDBOOKINGFORM_IF_NOT_REGISTERED and #_REMOVEBOOKINGFORM_IF_REGISTEREDYour logic is almost correct: you can ask for extrta info, but just not for “main info” (like #_LASTNAME etc ..). You can just ask for extra info for custom fields you created.
So create a field for extra lastnames, aother one for firstnames and call that one in your template (using #_FIELD{xx} syntax)If you have a “please wait” that remains it means you have a javascript error preventing the correct response from showing on screen (probably coming from a generated php warning in the aajax reply). Check your browser console for hints.
The mail method is not used when you go to the page “Send Mails”, so that change is not related.
The only reason you’re seeing a blank page is because you probably have javascript errors on that page. Check your browser console for hints.That function can’t work. You don’t assign anything to $calculated_discount …
That probably triggers a PHP warning, preventing the mail to be send.
