Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
countrycharm
All I was saying if everything was working fine for you in the first place why update. If you have read any of the
thread a few pages back before you posted you would have seen that there were some minor things updated. Again if everything was working you didn't need to update. I will update this module sometime this coming week with the new install.txt
Sounds like some good things are coming for this add-on.. I do think WebKat makes some good points and perhaps it makes sense for you giuys to work together on the next release to incorporate some of these ideas..:D
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
WebKat
By the way, I hope it isn't considered bad form, but if you don't want your customers' last names to show up on their testimonials (privacy issues) you can follow these edits that I made to my installation which makes it just show the first name:
File to edit:
/includes/modules/pages/testimonials_add/header_php.php
Replace the default code to this new code; line numbers may be approximate--make sure the one you're replacing is very similar to the replacement code:
Line 126
Code:
$account_query = "select customers_firstname, customers_email_address
Line 132
Code:
$extra_info=email_collect_extra_info($name,$email_address, $account->fields['customers_firstname'] . ' ' . $account->fields[''] , $account->fields['customers_email_address'] );
Line 150
Code:
$sql = "SELECT c.customers_id, c.customers_firstname, c.customers_email_address, c.customers_default_address_id, c.customers_telephone, ab.customers_id, ab.entry_street_address, ab.entry_company, ab.entry_city, ab.entry_postcode, ab.entry_zone_id, ab.entry_country_id, z.zone_id, z.zone_code, cn.countries_id, cn.countries_name FROM " . TABLE_CUSTOMERS . " c, " . TABLE_ADDRESS_BOOK . " ab, " . TABLE_ZONES . " z, " . TABLE_COUNTRIES . " cn WHERE c.customers_id = :customersID AND ab.customers_id = :customersID AND ab.entry_country_id = cn.countries_id";
Line 155
Code:
$testimonials_name = $check_customer->fields['customers_firstname'] . ' ' . $check_customer->fields[''];
That should do it. It should never call their last name in the first place, so you don't need to "hide" it in the displayed testimonial, since that bit of information isn't there to begin with.
I Will keep this in mind unless there is a demand for it. Since the original author wrote the module like this in the first place I think I will leave it as is. Thank you for the your input.
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
WebKat
I'm generally in the habit of keeping modules/plugins/scripts/etc. up-to-date with the most recent version. If I see an updated version has been released I generally update my copy to stay current. It's a lesson I learned the hard way a few times by failing to do so.
Quote:
Originally Posted by
DivaVocals
Sounds like some good things are coming for this add-on.. I do think WebKat makes some good points and perhaps it makes sense for you giuys to work together on the next release to incorporate some of these ideas..:D
Thanks you for your concern and support. Webkat you did have some valid points and thank you for bringing them up. I have created and upgrade folder and created a upgrade.txt file. Maybe this will be a lot easier for the installer. Thank Again
Waiting on zen to approve it in the downloads.
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
countrycharm
I Will keep this in mind unless there is a demand for it. Since the original author wrote the module like this in the first place I think I will leave it as is. Thank you for the your input.
You may want to put it in the install/upgrade/whatever text file as an optional modification. It seems like having your full name displayed on a testimonial might frighten off a lot of people who would otherwise write one, so it seems like the ability to set it to not display would be one a decent amount of people would like (who may not have thought of it before seeing it as an option). Several modules I've downloaded have had instructions for various tweak/hacks in their documentation, so it wouldn't be unusual.
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
WebKat
You may want to put it in the install/upgrade/whatever text file as an optional modification. It seems like having your full name displayed on a testimonial might frighten off a lot of people who would otherwise write one, so it seems like the ability to set it to not display would be one a decent amount of people would like (who may not have thought of it before seeing it as an option). Several modules I've downloaded have had instructions for various tweak/hacks in their documentation, so it wouldn't be unusual.
Great!!!:bigups:
Gonna say this again..:smile: YOU can also make these changes and submit them if you REALLY feel strongly that they should be a part of the add-on.. This would be an excellent place for you to make such a contribution!
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
WebKat
You may want to put it in the install/upgrade/whatever text file as an optional modification. It seems like having your full name displayed on a testimonial might frighten off a lot of people who would otherwise write one, so it seems like the ability to set it to not display would be one a decent amount of people would like (who may not have thought of it before seeing it as an option). Several modules I've downloaded have had instructions for various tweak/hacks in their documentation, so it wouldn't be unusual.
Quote:
Originally Posted by
DivaVocals
Great!!!:bigups:
Gonna say this again..:smile: YOU can also make these changes and submit them if you REALLY feel strongly that they should be a part of the add-on.. This would be an excellent place for you to make such a contribution!
Yes that OK with me. Feel free to update and make the change. Just wait until I let you know that when the new update has been approved.
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
BillyBoyle
@ Brian
you could use a blank sidebox available through the download section and set up a static page for your detailed testimonials. Putting a link in the sidebox to your page.
The blank sidebox worked great! Thank you!
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
stoneroses6300
The blank sidebox worked great! Thank you!
:bigups: your welcome.
Re: Testimonial Manager Support Thread
Quote:
Originally Posted by
WebKat
You may want to put it in the install/upgrade/whatever text file as an optional modification. It seems like having your full name displayed on a testimonial might frighten off a lot of people who would otherwise write one, so it seems like the ability to set it to not display would be one a decent amount of people would like (who may not have thought of it before seeing it as an option). Several modules I've downloaded have had instructions for various tweak/hacks in their documentation, so it wouldn't be unusual.
Quote:
Originally Posted by
DivaVocals
Great!!!:bigups:
Gonna say this again..:smile: YOU can also make these changes and submit them if you REALLY feel strongly that they should be a part of the add-on.. This would be an excellent place for you to make such a contribution!
Quote:
Originally Posted by
countrycharm
Yes that OK with me. Feel free to update and make the change. Just wait until I let you know that when the new update has been approved.
OK this update is now active in the downloads area. Webkat feel free to to do what you have suggested if you like to make such a contribution!
Re: Testimonial Manager Support Thread
Looks much better--much less confusing. I still made a few changes and sent you a PM with a download link to a new zip file.
