|
2-3 bugs 1 Year, 6 Months ago
|
Karma: 2
|
hi, guys, this is my first post here,
i want to say :
Congratulations for creators of tomatocart, is sow easy to work with this source & and hopefully will remain free for life
now i will report my bugs, also with the fixes
this will prevent to create another account when u are already logged & also if u are not logged u cannot see " My wish list" , sow, i have modified little the source , when u already logged & u try to make another accont u will be redirected to your account details & for " My wish list" u will be redirected to the Login page
here are instructions:
Open "wishlist.php" file from root/includes/content/account/ folder & find:
| Code: |
global $osC_Language, $osC_Services, $breadcrumb, $messageStack;
|
replace with:
| Code: |
global $osC_Language, $osC_Services, $breadcrumb, $messageStack, $osC_Customer;
|
find :
| Code: |
$this->_page_title = $osC_Language->get('wishlist_heading');
|
replace with:
| Code: |
if ($osC_Customer->isLoggedOn()) {
$this->_page_title = $osC_Language->get('wishlist_heading');
}else{
osc_redirect(osc_href_link(FILENAME_ACCOUNT, 'login', 'SSL'));
}
|
Next step:
Open "create.php" file from root/includes/content/account/ folder & find:
| Code: |
global $osC_Language, $osC_Services, $breadcrumb;
|
replace with:
| Code: |
global $osC_Language, $osC_Services, $breadcrumb, $osC_Customer;
|
find:
| Code: |
$this->_page_title = $osC_Language->get('create_account_heading');
|
replace with:
| Code: |
if (!$osC_Customer->isLoggedOn()) {
$this->_page_title = $osC_Language->get('create_account_heading');
}else{
osc_redirect(osc_href_link(FILENAME_ACCOUNT, 'account', 'SSL'));
}
|
U can prevent also directly from templates files
open "index.php" from "glass_grey" template
find:
| Code: |
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'wishlist', 'SSL'), $osC_Language->get('my_wishlist')); ?>
</li>
<?php if ($osC_Customer->isLoggedOn()) { ?>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'logoff', 'SSL'), $osC_Language->get('logoff')); ?>
</li>
<?php } else { ?>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'login', 'SSL'), $osC_Language->get('login')); ?>
</li>
<?php } ?>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'create', 'SSL'), $osC_Language->get('create_account')); ?>
</li>
|
Replace with:
| Code: |
<li>
<?php if ($osC_Customer->isLoggedOn()) { ?>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'wishlist', 'SSL'), $osC_Language->get('my_wishlist')); ?>
</li>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'logoff', 'SSL'), $osC_Language->get('logoff')); ?>
</li>
<?php } else { ?>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'login', 'SSL'), $osC_Language->get('login')); ?>
</li>
<li>
<?php echo osc_link_object(osc_href_link(FILENAME_ACCOUNT, 'create', 'SSL'), $osC_Language->get('create_account')); } ?>
</li>
|
I hope to help my fixes
i will be back with other wen i will find
Best regards, hack346 |
|
|
|
I convert (create) themes for Tomatocart |
|
|
Re:2-3 bugs 1 Year, 6 Months ago
|
Karma: 12
|
I'm not sure to agree if these are any bugs, seems to me to be by design.
Why would you want to allow to create a new account when you are already logged in?
And for wish list, well, that depends on if it will save in a cookie even if you are not logged in and if it does it for a long time.
And if you have it and then create an account it will save in your account. Does it work like this after your bug fix? If so I agree it's a bug fix and it's good  |
|
|
|
|
|
|
Re:2-3 bugs 1 Year, 6 Months ago
|
Karma: 2
|
Multiple account for a serious sales company?, i don`t think ...realy i don`t understand why a person will try to make another account when already is logged in & about my wish list ...& cookies? not work on windows servers, i will test later on linux  |
|
|
|
Last Edit: 2010/07/29 11:30 By hack346.
I convert (create) themes for Tomatocart |
|
|
Re:2-3 bugs 1 Year, 6 Months ago
|
Karma: 12
|
Oh I misread  . You fixed so you CAN'T create a new account when already logged in, that's good 
I thought you did the opposite ^^
For wishlist, I think it's good that it works even if you are not logged in. |
|
|
|
|
|
|
Re:2-3 bugs 1 Year, 6 Months ago
|
Karma: 29
|
|
Hi, thanks for the bug report. It is correct that a user logged in should not create an account again.
Concerning the wishlist it is a feature, we want to enable the customers not logged in also can use wishlist and send wishlist to their friends.
|
|
|
|
|
|
|
Community
