Jump to content

nfrobertson

Members
  • Content count

    50
  • Joined

  • Last visited

Profile Information

  1. nfrobertson

    Is there a support thread for SID Killer ?

    seo.class.php is an impressively large bit of code. I see there is a new 2.2.2 out there which includes fixes for mysql5. Right now I am still running mysql4 on my shop and don't have the need to upgrade. Honestly, I spent many many hours tracing through the code last time and testing my tweaks to get the specific version 2.2.1e to work "just ok" with SID killer. I'm guessing it'll take a similar amount of time with the new version. I also know that Chemo has his own official version of seo.class.php out there (not on this site) and it is different than the 2.2.1e that I used for my hack with SID killer here. All that being said, I'm sorry I don't have the bandwidth right now to further pursue SEO and SID killer. I'm actually hoping that the oscommerce team tackles these issues in a new release of the software baseline since all the main players who wrote this stuff are now gone. Nathan
  2. nfrobertson

    Register Globals Support

    Wow, I just tried this and sure enough - works as you describe (which is not what is desired) I just got done testing tell_a_friend.php a coupld days ago and it was working great. I then loaded the Register Globals patch. I thought everything seemed to be working fine but guess I hadn't really noticed this yet. Is there a fix?? Nathan
  3. Ok. It didn't seem to hard to hack on checkout_payment.php so here's what I did to get the image to now show up. In checkout_payment.php about line 285 <?php } elseif (isset($selection[$i]['fields']) && is_array($selection[$i]['fields'])) { add this code right after the above line so it checks if image is set and includes the HTML if (isset($selection[$i]['image'])) { ?> <tr> <td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td> <td class="main" colspan="4"><?php echo $selection[$i]['image']; ?></td> <td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td> </tr> <?php } I simply modeled this on the code that handled the error case above on about line 278. This hasn't been extensively tested yet... Is there a better way or another contrib that did something like this that TADD may have had enabled? Nathan
  4. I tried this but didn't get the image to show up. I took a look in checkout_payment.php and I see where it uses $selection[$i]['id'] $selection[$i]['module'] $selection[$i]['error'] $selection[$i]['fields'] but I can't find any mention of $selection[$i]['image'] Is there another contrib or more code that has to be added to get this to work? Thanks! Nathan
  5. I just gave this a shot and it works great! :thumbsup: -Tada indeed! I threw in some apache trigger_errors to see the values (before and after) this fix: BEFORE [17-Jun-2006 17:41:18] PHP Notice: last_order_id : Resource id #56 in ..../includes/modules/payment/authorizenet_aim.php on line 229 [17-Jun-2006 17:41:18] PHP Notice: new_order_id1 : in ..../includes/modules/payment/authorizenet_aim.php on line 231 [17-Jun-2006 17:41:18] PHP Notice: new_order_id2 : 1 in ..../includes/modules/payment/authorizenet_aim.php on line 233 AFTER [17-Jun-2006 17:43:28] PHP Notice: last_order_id FIX : Array in ..../includes/modules/payment/authorizenet_aim.php on line 240 [17-Jun-2006 17:43:28] PHP Notice: new_order_id FIX : 5016 in ..../includes/modules/payment/authorizenet_aim.php on line 242 Order #5016 was indeed the next order number. I checked the email from Auth.net and it was listed as expected! I did this in Auth.net test mode. Assume it will work the same in non-test mode. Vger, can this be added to the official release of this module? Thanks.
  6. nfrobertson

    [Support] Simple Down for Maintenance

    Hello. This contrib is exactly what I was looking for. I'm using STS and immediately ran into the problem with this contrib and STS. I also looked at the 4/19/06 addition by betacodehex to fix this issue. I think I have a simpler way and would like to see what you think. In includes/header.php, instead of inserting the PHP wrapped HTML, why not add the DOWN_FOR_MAINTENANCE_MESSAGE to the list of warnings that are already built into OSC? Here's the code I use to do that. This appears to work with and without STS. Add if (DOWN_FOR_MAINTENANCE_WARNING == 'true') { $messageStack->add('header', DOWN_FOR_MAINTENANCE_MESSAGE, 'warning'); } just after the last Warning check. Below is what it should look like after you're done. I've snipped out extra code to shorten the listing. // check if the 'install' directory exists, and warn of its existence ...snip... // check if the configure.php file is writeable ...snip... // check if the session folder is writeable ...snip... // check session.auto_start is disabled ...snip... if ( (WARN_DOWNLOAD_DIRECTORY_NOT_READABLE == 'true') && (DOWNLOAD_ENABLED == 'true') ) { if (!is_dir(DIR_FS_DOWNLOAD)) { $messageStack->add('header', WARNING_DOWNLOAD_DIRECTORY_NON_EXISTENT, 'warning'); } } if (DOWN_FOR_MAINTENANCE_WARNING == 'true') { $messageStack->add('header', DOWN_FOR_MAINTENANCE_MESSAGE, 'warning'); } if ($messageStack->size('header') > 0) { echo $messageStack->output('header'); } Thoughts? Also, it would be helpful if the SQL was also included in a .sql file inside the contrib. I know it's not required but would be helpful to run it all at once. Thanks Nathan
  7. nfrobertson

    New USPS shipping server address?

    I have also received the letter on 4/27/06. I tried this quick test, chaning the server name, but it hung and returned nothing. I'm really hoping that we don't have to switch from http://production.shippingapis.com to https://secure.shippingapis.com Anyone else heard any more on this? Nathan
  8. nfrobertson

    USPS web APIs requiring HTTPS

    I have the same question, any luck finding an answer? Nathan
  9. nfrobertson

    [Contribution] - USPS Methods

    I've been using the USPSMethods 2.8d module for a few months now successfully. On 4/27/06 I received an email from USPS that they will being requiring HTTPS connections starting in September 30, 2006 and the old HTTP connections to http://production.shippingapis.com will go away. Has anyone addressed the changes necessary to update USPSMethods 2.8d to go to: https://secure.shippingapis.com Your help is much appreciated. Nathan
  10. nfrobertson

    Country-State Selector

    stevel, Thanks for the code updates. Sorry it took me a while to get them implemented. So far, initial testing is good so it seems to work as I'd expect. Again, thanks!! Nathan
  11. We discussed this topic in the thread for my "Encrypt Credit Card with mcrypt" contrib. You can find the thread by Clicking Here In that discussion a person didn't want the encryption for just the same reason (auth.net saves it) The example sets all credit card numbers in the db to the format xxxxxxxxxxxx1234. You could modify the example though to come up with 1234xxxxxxxx1234.
  12. nfrobertson

    Country-State Selector

    First - EXCELLENT contribution! This was the first one I installed on my long trek to getting my oscommerce shop up and running. I finally got around to figuring out my tax setup (bear with me, this really does relate to Country-State Selector) Anyway, I'm in Colorado so I have to care about sales tax in my own county OR in all other counties in the state. After reviewing how other people have setup taxes for counties in a state I have now created two tax zones: zone_id zone_country_id zone_code zone_name 13 223 CO Colorado 79 223 CO Colorado (Douglas County) Everything was going along swimmingly until I tried to use the Change Address in either the shipping page or billing page. I kept getting the error "Please select a state from the States pull down menu." After some review, I determined this was happening in checkout_shipping_address.php (approx line 111) and checkout_payment_address.php (approx line 98). The code that selects the zone_id is having fits because of my similar beginning zone_names above (Colorado% will match both zone_names). So, I figure that since this is in a section of code that knows there are zones, and sinze I'm using Country-State Selector I don't really need the matching logic in the SQL that's included. If I recall correctly, this type check was replaced/removed by this contribution in the create_account.php file. I changed $zone_query = tep_db_query("select distinct zone_id from " . TABLE_ZONES . " where zone_country_id = '" . (int)$country . "' and (zone_name like '" . tep_db_input($state) . "%' or zone_code like '%" . tep_db_input($state) . "%')"); to be $zone_query = tep_db_query("select distinct zone_id from " . TABLE_ZONES . " where zone_country_id = '" . (int)$country . "' and zone_name = '" . tep_db_input($state) . "'"); It seems to me that this edit could be safely included in the next release of Country-State Selector? Does anyone see any problems with this change?
  13. I have a couple questions about the process_button() function function process_button() { // Hidden fields on the checkout confirmation page $process_button_string = tep_draw_hidden_field('cc_owner', $_POST['authorizenet_aim_cc_owner']) . tep_draw_hidden_field('cc_expires', $this->cc_expiry_month . substr($this->cc_expiry_year, -2)) . tep_draw_hidden_field('cc_type', $this->cc_card_type) . tep_draw_hidden_field('cc_number', $this->cc_card_number); if (MODULE_PAYMENT_AUTHORIZENET_AIM_USE_CVV == 'True') { $process_button_string .= tep_draw_hidden_field('cc_cvv', $_POST['authorizenet_aim_cc_cvv']); } $process_button_string .= tep_draw_hidden_field(tep_session_name(), tep_session_id()); return $process_button_string; return false; } 1) Why are there two returns from this function? The second isn't reachable. Just a cleanup item? 2) Why is the tep_session added as a hidden field? All the other payment modules, except for the default Authorize.net (SIM) and this (AIM) module do not set the tep_session into the string. The reason I'm asking about the tep_session is that this causes the checkout_success.php page to show the SID in the URL. I would like to avoid the SID in the URL unless it's really needed. Thoughts? Thanks!
  14. Sorry. QBI is the QuickBooks Import contrib by Adam Liberman. You can find the contrib HERE and the support forum HERE (see the last page). HERE is my post regarding the issue between these two contribs. Anyway, basically is comes down to the variable that the default payment modules define for their "*TEXT_TITLE" that QBI looks for. Your contrib modifes that slightly to *TEXT_ADMIN_TITLE and *TEXT_CATALOG_TITLE. I worked around the issue via my last post by adding the definition of the *TEXT_TITLE also... // Admin Configuration Items define('MODULE_PAYMENT_AUTHORIZENET_AIM_TEXT_TITLE', 'Authorize.net AIM'); // FIX for QBI not "seeing" Authorize.net AIM define('MODULE_PAYMENT_AUTHORIZENET_AIM_TEXT_ADMIN_TITLE', 'Authorize.net AIM'); // Payment option title as displayed in the admin It seems that this could be added to the next version of Authorize Net AIM (GPL) and that will keep it compliant with the other payment modules. Thoughts?
  15. Vger, I found an issue between Authorize Net AIM (GPL) and the QBI contrib. Click Here to read about it. Basically, QBI is scanning the payment modules language files for the partial string "TEXT_TITLE" which all the default osc payment modules have. Authorize Net AIM (GPL) has implemented different titles for the admin area vs catalog area using variable slightly different from the default payment modules. To work around this, I just added another define to the languages/english/modules/payment/authorizenet_aim.php file which looks like: // Admin Configuration Items define('MODULE_PAYMENT_AUTHORIZENET_AIM_TEXT_TITLE', 'Authorize.net AIM'); // FIX for QBI not "seeing" Authorize.net AIM define('MODULE_PAYMENT_AUTHORIZENET_AIM_TEXT_ADMIN_TITLE', 'Authorize.net AIM'); // Payment option title as displayed in the admin Do you see any problems doing this? Since your contrib doesn't use that variable I'm hoping it's not an issue. Maybe this could be added to a future version of Authorize Net AIM (GPL) to keep it compatible with QBI? Thanks for your help.
×