X-Cart forums

[jmccunep]

It appears there is possibly a coding error in latest reBOOT, a minor one. When customers submit contact-us forms the subject lines they use are overwritten, replaced by the same generic subject line every time.

Maybe the code in this template is off? reboot/customer/help/contactus.tpl

Perhaps something wrong related to this variable? $smarty.get.subject

Could this be a problem only with our installation of reBOOT (and reCAPTCHA) which you installed for us, Phil.

X-cart 4.7.9 with matching reBOOT template.

[PhilJ]

In skin/reboot/customer/help/contactus.tpl

You can replace...

<input type="text" class="maxlength" maxlength="100" id="subject" name="subject" size="32" maxlength="128" value="{if $smarty.get.subject}{$smarty.get.subject}{else}{$userinfo.subject|escape}{/if}" />

With...

<input type="text" class="maxlength" maxlength="100" id="subject" name="subject" size="32" maxlength="128" value="{$userinfo.subject|escape}" />

[ITVV]

Hi Phil,

Sorry to trouble you as I know that you are busy with reDux

I am having lots of issues with reboot dynamic search.

I have this logged as an error: -

[05-May-2018 08:45:19 Europe/London] PHP Fatal error:  Uncaught Error: Call to undefined function mysql_connect() in /home/fgfrrtfx/public_html/website.com/reboot_dynamic_search.php(1) : eval()'d code(1) : eval()'d code(1) : eval()'d code:6
Stack trace:
#0 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search.php(1) : eval()'d code(1) : eval()'d code(1): eval()
#1 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search.php(1) : eval()'d code(1): eval()
#2 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search.php(1): eval()
#3 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search_queries.php(1) : eval()'d code(1) : eval()'d code(1) : eval()'d code(3): include_once('/home/accfcfbf/...')
#4 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search_queries.php(1) : eval()'d code(1) : eval()'d code(1): eval()
#5 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search_queries.php(1) : eval()'d code(1): eval()
#6 /home/accfcfbf/public_html/itvv.com/reboot_dynamic_search_queries.php(1): eval()
#7 {main}
  thrown in /home/fgfrrtfx/public_html/website.com/reboot_dynamic_search.php(1) : eval()'d code(1) : eval()'d code(1) : eval()'d code on line 6

I beleive that it is caused because we are using PHP 7? The error referes to 'mysql_connect()' which should be 'mysqli_connect()' for PHP 7 if I'm not mistaken?

I cannot change this because the file is encoded.

Could you be so kind as to post an updated reboot_dynamic_search.php file please?

Would any other files need patching for search to work as it used to?

Many thanks and kind regards as always

ITVV

PS Looking forward to seeing all your hard work on reDux

[PhilJ]

Hiya, post a ticket and I'll attach you the PHP7 compatible files. Or you can get updates from here. We no longer encode any files since a few years back.

[ITVV]

WOW Thanks Phil

All sorted

BRILLIANT SUPPORT AS ALWAYS

Saturday afternoon and got a response within 25 mins of posting a request.

Many many thanks Phil

Kind regards from a very happy customer

ITVV

[jmccunep]

Originally Posted by PhilJ

In skin/reboot/customer/help/contactus.tpl

You can replace...

<input type="text" class="maxlength" maxlength="100" id="subject" name="subject" size="32" maxlength="128" value="{if $smarty.get.subject}{$smarty.get.subject}{else}{$userinfo.subject|escape}{/if}" />

With...

<input type="text" class="maxlength" maxlength="100" id="subject" name="subject" size="32" maxlength="128" value="{$userinfo.subject|escape}" />

For the above contact-us template code, it would help our customer support staff if a unique identifier could be auto-appended (auto-pre-pended actually) to the customer-entered subject line. The identifier could be for example a date-time snippet or numeric counter number.

Can anyone suggest code (Smarty? PHP?) that would accomplish this so that the result of the subject value would be changed, conceptually, as follows?

value="[pre-pended counter or date-time or other unique identifier][space]{$userinfo.subject|escape}"

[PhilJ]

You could use {$smarty.now} in the subject line as a unique numeric identifier (unix timestamp).

Edit skin/common_files/mail/help_contactus_subj.tpl

Eg.

{config_load file="$skin_config"}{$config.Company.company_name}: {$lng.eml_contact_us_subj} | {$smarty.now} | {$smarty.now|date_format:$config.Appearance.date_format} | {$smarty.now|date_format:$config.Appearance.time_format} | {$contact.firstname} {$contact.lastname} | {$contact.email} | {$contact.subject}

And/or add the necessary code to skin/reboot/mail/html/help_contactus.tpl

[jmccunep]

Originally Posted by PhilJ

You could use {$smarty.now} in the subject line as a unique numeric identifier (unix timestamp).

Edit skin/common_files/mail/help_contactus_subj.tpl

Eg.

{config_load file="$skin_config"}{$config.Company.company_name}: {$lng.eml_contact_us_subj} | {$smarty.now} | {$smarty.now|date_format:$config.Appearance.date_format} | {$smarty.now|date_format:$config.Appearance.time_format} | {$contact.firstname} {$contact.lastname} | {$contact.email} | {$contact.subject}

And/or add the necessary code to skin/reboot/mail/html/help_contactus.tpl

Thanks for being willing to look at this!

Would this code be accurate/correct then and would it append the date & time to the end of the subject line?

{config_load file="$skin_config"}{$config.Company.company_name} : {$lng.eml_contact_us_subj}{$smarty.now|date_format :'%Y-%m-%d %H:%M:%S'}

To put the date & time BEFORE the subject would this be accurate?

{config_load file="$skin_config"}{$config.Company.company_name} : {$smarty.now|date_format:'%Y-%m-%d %H:%M:%S'}{$lng.eml_contact_us_subj}

Oh, should insert a space as well between subject and date. Sorry, Smarty newbie, don't now basics how to insert literals...

[PhilJ]

Put what you like in there

[PhilJ]

Hiya, GDPR files attached for reBOOT v4.7.9. Get the GDPR module files for your version here.

Any problems, please let me know.

Always a good idea to get yourself a decent file comparison tool - KDiff - WinMerge - Beyond Compare - etc.

Thanks for your patience re. reDUX, temporary demo here.