swilders

Found the issue at our side. We have an action hook of EmailPreSend which checks the expiry date of a domain and aborts the send if the actual expiry date is greater than the number of days on which our first domain renewal is sent. We set this up because in the past we had renewal reminders going out for domains that had already been renewed.

We haven't seen any reminders go out since the end of March. Also, when trying to manually send we get: An Error Occurred Email sending aborted.

I ended up adding some jQuery to header.tpl to check the box for us: I changed this... <script type="text/javascript"> var datepickerformat = "{$datepickerformat}"; {if $jquerycode}$(document).ready(function(){ldelim} {$jquerycode} {rdelim}); {/if} {if $jscode}{$jscode} {/if} </script> to this... <script type="text/javascript"> var datepickerformat = "{$datepickerformat}"; {if $jquerycode}$(document).ready(function(){ldelim} $("input[type='checkbox'][name^='taxed']").each(function() {ldelim} $(this).prop('checked',true); {rdelim}); $("input[type='checkbox'][name^='taxable']").each(function() {ldelim} $(this).prop('checked',true); {rdelim}); {$jquerycode} {rdelim}); {/if} {if $jscode}{$jscode} {/if} </script> Works a treat. Just be careful not to overwrite the change when upadting if editing one of the default templates (like I've done on several occasions!) NB: on quotes, it checks the box after saving each line.

Hello, It would be useful if a quote could have a field for markup in the admin area which I can then pull into reports.

Hello, When creating a new invoice the "Taxed" checkbox is ticked by default. On a quote it isn't. Please can we have it ticked by default on a quote?

Thanks Matt. We're running 5.2.12. I just wanted to be 100% sure. Thanks.

Hello, I've applied the last 2 security patches as soon as I received notification about them. This morning I found a new signup on our WHMCS install and the log shows the following: Client Profile Modified - First Name: 'naksdnkas' to 'AES_ENCRYPT(1,1), firstname= (SELECT GROUP_CONCAT(id,0x3a,username,0x3a,email,0x3a,password SEPARATOR 0x2c20) FROM tbladmins)', Last Name: 'asdasdas' to '1', Company Name: 'asdasdasd' to '1', Address 1: 'asdasdasd' to '1', Address 2: 'asdasd' to '1', City: 'sadasd' to '1', State: 'Bath and North East Somerset' to '1', Postcode: '56 564' to '1', Country: 'GB' to 'US', Phone Number: '435345345' to '1', Default Payment Method: '' to '' Have I anything to worry about or do the patches prevent this? I have ensured all admins have changed their passwords to be on the safe side.

Hello, When I use the getproducts function of the API I don't get a return of all the products. From what I can tell the return is random, there's a mixture of hidden and visible products, as well as products that are archived and products that are not - how can I get a list of ALL products using the API?

Hello, I can assure you my code is sound, feel free to try for yourself using my steps to reproduce the issue. The example I have given is simply quoting a print_r of the returned array.

Furthermore, the WHMCSAPI:TOTALRESULTS returns 0 even when there are results.

Hello, I don't think the getquotes part of the API is working correctly. If I request quotes for a client that has multiple quotes in WHMCS then the WHMCSAPI:QUOTES:QUOTE:ITEMS section of the returned data appends an increment to the end of the array keys, and also skips a number in between. So, for example, on the first quote I will get: WHMCSAPI:QUOTES:QUOTE:ITEMS:ITEM WHMCSAPI:QUOTES:QUOTE:ITEMS:ITEM1 WHMCSAPI:QUOTES:QUOTE:ITEMS:ITEM2 WHMCSAPI:QUOTES:QUOTE:ITEMS:ITEM3 WHMCSAPI:QUOTES:QUOTE:ITEMS:ITEM4 and then the next quote is returned as: WHMCSAPI:QUOTES:QUOTE:ITEMS6:ITEM7 WHMCSAPI:QUOTES:QUOTE:ITEMS6:ITEM8 WHMCSAPI:QUOTES:QUOTE:ITEMS6:ITEM9 WHMCSAPI:QUOTES:QUOTE:ITEMS6:ITEM10 etc This make it very difficult for me to iterate through the returned items. I can get around the issue because the pattern is at least predictable if I count the items and add 1 between quotes, but that's hardly ideal - is this intentional or an error?

Hello, I did get this working in the end. It might not be the most elegant solution but $vars doesn't include the required information to make the decision without a database query. This works fine for us though: function domain_expiry_2($vars) { $merge_fields=array(); if($vars['relid']&&$vars['messagename']=='Upcoming Domain Renewal Notice') { $daysuntilrenewal=mysql_fetch_array(mysql_query("SELECT DATEDIFF(expirydate,'".date("Y-m-d")."') FROM tbldomains WHERE id=".$vars['relid'])); if($daysuntilrenewal[0]>=60) $merge_fields['abortsend']=true; } return $merge_fields; } add_hook("EmailPreSend",1,"domain_expiry_2");

No but it looks to have been corrected now as the quotes do now show the product group and order as described above.

Our clients pay us on 30 day terms. They get a domain renewal reminder, they confirm they want it renewing so we renew it and invoice them. They pay in approximately 30 days time. The problem with WHMCS is that even though it is renewed, it still sends domain renewal reminders until it is paid. These renewals look silly anyway because they say, for example, that the domain is going to expire in 737 days! The reminders should be based on the domain expiry date not the next due date. WHMCS advised us to write an action hook to stop the email is this situation. We've tried to set this up but This doesn't seem to work. function domain_expiry_2($vars) { $daysuntilrenewal=$vars['domain_days_until_expiry']; $merge_fields=array(); if($daysuntilrenewal>=60) $merge_fields['abortsend']=true; return $merge_fields; } add_hook("EmailPreSend",1,"domain_expiry_2"); Please can you quote to write a working version of this.

Hi, I installed the project management add-on yesterday and when I type in the "Associated client" field the auto-complete simply displays "ACCESS DENIED". Any ideas?