brian!

I hope you can see the hole in that suggestion. 🙄

of course, whilst trying to avoid those developers on there who may have some fake reviews!

oh ho... I've warned for years against using the bulk pricing updater... I wouldn't even play with it in a dev, and I definitely wouldn't use it with domain pricing (which I know is not what you used it for). not from settings... though I daresay you could use the UpdateClietnProduct API to do it by looping through a list of service IDs (as it includes an autorecalc option that you can set to true). I assume you mean all services as opposed to all clients... the existing "Services" report could be quickly tweaked to include promo ID values which you can then search for... or a quick SQL query such as the one below would give you a list of services with promotions... SELECT tblhosting.* FROM tblhosting WHERE tblhosting.promoid > 0 possibly the Auto Recalculate Prices addon can do this too, but it's description makes no reference to promotion codes, so i'd suggest checking on that point first before buying it.

I think we're going to have to see your hook code to see exactly what it's doing.

this is the ServerPing module by Joe Clarke ?? there's a protected download update on their site (no date on it, but I suspect it's not relevant to v7.8), but I know the product itself is no longer offered on site... perhaps it's just being pulled while they work out any kings with v7.8, or maybe it's just been abandoned... might be worth contacting them to confirm. there are a couple in Marketplace, but they're not marked as compatible with v7.8 yet - though that could just be an oversight in not updating the listings, or they may be incompatible...

Hi @pjs32 apologies for the delay in replying - got numerous paid projects on the go and plenty of domestic distractions! as suspected, the first half was straightforward (always showing all available renewal periods); but the second half (your issue) is proving trickier to resolve.. even if you modify the array using a hook, it seemingly isn't pulling pricing values from that array when you add a renewal to the cart, it's pulling it from domain pricing... so visibly, you can override the pricing, but not the price when you add it to the cart... of course, the override hooks don't work with renewals, so they're not an option. I can see one way to do it, but it's a solution that might not necessarily work for everyone... a rough beta is currently being tested for me... I expect the first half to be released as a hook soon (just needs a little polishing); whereas, I think the second half may need to be more than a hook, or I need to think of the most appropriate way for it to work.

the 151, 204 & 224 could well be correct for the outer if statement, but they are the configurable option values (.eg Quantity (GB), Server Branding etc)... however, each option within a configurable option select dropdown has it's own value and it's that value that you need to use in this inner if statement.

then I suspect you're using the wrong value(s) for $option.id - let's use the "Server Branding" option below... <td> <select name="configoption[36]"> <option value="67" selected>No Change</option></option> </select> </td> so from that above code (viewed from the browser), the value that you want is not "36" from the configoption, it's "67" - with that value, if I then modify the template... {if $option.selected}<option value="{$option.id}" selected>{if $option.id eq '67'}Brian{else}{$LANG.upgradenochange}{/if}</option>{else}<option value="{$option.id}">{$option.nameonly} {$option.price}{/if}</option> ignore that i'm hardcoding the output and I haven't included your outer IF statement, I get the following... so I suspect you're going to have to work your way through viewing the source of the page in the browser and finding what the three values are for these specific "No Change" options and use *those* values in your inner IF statement.

i'm confused - the above debug window shows that, in these domain templates, you can access the variable via Smarty in the email template itself... 😕 so I guess that we've reached the stage where we need to ask - what you want to do with this domain ID value?

I suspect that... {if $option.id eq '151' and $option.id eq '204' and $option.id eq '224'} should really be... {if $option.id eq '151' or $option.id eq '204' or $option.id eq '224'} the point being that you are using this code whilst inside a loop, so one of those conditions can be true, but never all three at the same time... e.g the current option is either 151, 204 or 224 - it can't be all three at the same time (which is what your code is currently checking for).

I suspect it would have to be an {if} statement in the upgrade.tpl template (as that is where the string is used, not in an array)... untested, but I think something along the lines of.. {if $option.selected}<option value="{$option.id}" selected>{$LANG.upgradenochange}</option>{else}<option value="{$option.id}">{$option.nameonly} {$option.price}{/if}</option> becomes... {if $option.selected}<option value="{$option.id}" selected>{if $option.id eq 'x'}{$LANG.upgradesupport}{else}{$LANG.upgradenochange}{/if}</option>{else}<option value="{$option.id}">{$option.nameonly} {$option.price}{/if}</option> where 'x' is the ID value of the configurable option.

there are multiple commercial addons that can do that, e.g one would be Bonus For Funds - though i'm not sure if the bonus is automated or if you have to manually decide when to add it... the developers will know, so they'd be the ones to ask on that. there was a hook that was posted a couple of years ago that prevented the Add Funds emails from being delivered to the client... I suppose in your case you could modify the hook to disable sending only if it's add funds AND there isn't a discount/negative line in the same invoice (if you're wanting to manually edit the invoice after creation)...

it should work - unless you are editing a wrong .css file, e.g not the one in the active template... /* ***************************************************** ** Custom Stylesheet ** Any custom styling you want to apply should be defined here. ***************************************************** */ section#home-banner { background-color: red !important; }

sure, it's just a case of changing the output below to suit your needs... if ($issue->enddate) { $bodyhtml .= '<a href="serverstatus.php" class="list-group-item"><small><b>'.$issue->title.'</b> <label class="label" style="background-color: lightgrey; color: black">'.$issue->status.'</label> <label class="label" style="background-color: '.$prioritycolor.'">'.$issue->priority.'</label><br><span style="color: black">'.$issue->description.'</span><br>'.Lang::trans('networkissuesstatusscheduled').': '.fromMySQLDate($issue->startdate, true, true).' - '.fromMySQLDate($issue->enddate, true, true).'</small></a>'; } elseif ($issue->lastupdate) { $bodyhtml .= '<a href="serverstatus.php" class="list-group-item"><small><b>'.$issue->title.'</b> <label class="label" style="background-color: lightgrey; color: black">'.$issue->status.'</label> <label class="label" style="background-color: '.$prioritycolor.'">'.$issue->priority.'</label><br><span style="color: black">'.$issue->description.'</span><br>'.Lang::trans('networkissueslastupdated').' - '.fromMySQLDate($issue->lastupdate, true, true).'</small></a>'; } so if we remove the description from the output... if ($issue->enddate) { $bodyhtml .= '<a href="serverstatus.php" class="list-group-item"><small><b>'.$issue->title.'</b> <label class="label" style="background-color: lightgrey; color: black">'.$issue->status.'</label> <label class="label" style="background-color: '.$prioritycolor.'">'.$issue->priority.'</label><br>'.Lang::trans('networkissuesstatusscheduled').': '.fromMySQLDate($issue->startdate, true, true).' - '.fromMySQLDate($issue->enddate, true, true).'</small></a>'; } elseif ($issue->lastupdate) { $bodyhtml .= '<a href="serverstatus.php" class="list-group-item"><small><b>'.$issue->title.'</b> <label class="label" style="background-color: lightgrey; color: black">'.$issue->status.'</label> <label class="label" style="background-color: '.$prioritycolor.'">'.$issue->priority.'</label><br>'.Lang::trans('networkissueslastupdated').' - '.fromMySQLDate($issue->lastupdate, true, true).'</small></a>'; }

have you checked that by testing it? just because it isn't listed in the merge field list at the bottom of the template, doesn't mean that it doesn't exist - that's not a complete list! the above is from the "Expired Domain Notice" email template and $domain_id clearly exists and contains the correct ID value for that domain. note I would expect $domain_id to only exist in Domain Type email templates... if you're searching for the domain ID value in another email template type, then you may have to query the db based on the variables that you do have access to in that template.