Jump to content

piernas

Members
  • Content count

    697
  • Joined

  • Last visited

  • Days Won

    17

Everything posted by piernas

  1. Today I decided I got angry for the last time updating a table rate based module. Afer several tryouts to change the shipping rates just to see I had a typo in the values string -damned colons- and having to manually calculate the net prices for each row from the gross that came from the table provided by the courier... I had enough and modified the table rate module to show a real table where you can see what you're doing. Uses some javascript to update net/gross, add or remove rows and do some basic checks. It can be adapted to any other module that uses the same system. Note: The net pice is caclulated based on the tax class from the module. It will apply only to the countries in the module's zone and it's not savet to database but calculated on the fly. For this column to work you need to set up the tax rate first, save and enter the module configuration again. Here is the file: https://apps.oscommerce.com/wyYLv&sgipping-table-module-update @burt you can replace the stock one if you consider it fits the community project.
  2. piernas

    Shipping table is now a real table :)

    Thanks for testing. I'll update it.
  3. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    @marokech please download latest from here: https://apps.oscommerce.com/Imrlq&ultimate-seo-urls-rewritter-for-phoenix and please post any problem here:
  4. piernas

    Shipping table is now a real table :)

    I'll try to update the ones I've made in the past while I have some free time
  5. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    I don't have a frozen test site anymore and I changed the code for phoenix, so can't do that for now. I'll do once I finish other work in progress; anyway I found google defaults to english if x-default is not set so your site should be indexed already.
  6. piernas

    Shipping table is now a real table :)

    Updated for Phoenix: https://apps.oscommerce.com/2iSYY&editable-shipping-table-phoenix
  7. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Yes you'll just have to add it to the $alternate variable.
  8. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Hi David, I dodn't know about that tag - I'll add it in the future once the app has been tested.
  9. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Final version for CE Phoenix 1.0.7.10+ attached. I will open a new topic (this one is very old and almost nothing relates to the currrent app due to the huge amount of changes and fixes). I will upload it to the addons page soon, too. I didn't receive any feedback other than from @Omar_one. Does nobody consider this app of interest anymore? @zeeshop yes it works on phoenix 1.0.7.12. Please download new package attached and see the readme included for instructions. Ultimate_SEO_Urls v3.0.0.zip
  10. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Thank you, then I uploaded the stock one. It should contain these lines, I'll fix it on the final upload: case 'write.php': $oscTemplate->addBlock('<link rel="canonical" href="' . tep_href_link($PHP_SELF, 'products_id=' . (int)$_GET['products_id'], 'SSL', false) . '" />' . PHP_EOL, $this->group); break;
  11. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    @omar the new ht_canonical should be slightly different. It adds canonical tags to write review (review.php) that is currently rewritten to review.html. It's not important as this page should not be indexed by spiders but I think I should also add it to noindex pages. I want to study the new pages system to see if it's good to add rewrite rules to it, too, so / info.php?pages_id=4 would become /slug.html
  12. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Thank you, I'll take a look at those errors. Does the app work?
  13. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    I've almost finished this app. Still want to make a couple of tweaks and add proper instructions but all seems to work well. I've attached it in case someone wants to test it. No core changes, tested with Phoenix 1.0.7.10. Just upload the files and go to admin->configuration->Ultimate Seo Urls and enable the app. Feedback will be appreciated. USU3_beta.zip
  14. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Hi @ecartz, I've found an issue with a core module (nb_shopping_cart.php). For constructing the URI the module uses this code: // nb_shopping_cart.php (line 19): echo sprintf(MODULE_NAVBAR_SHOPPING_CART_PRODUCT, $p['id'], $p['quantity'], $p['name']); // languages/english/modules/navbar/nb_shopping_cart.php: define('MODULE_NAVBAR_SHOPPING_CART_PRODUCT', '<a class="dropdown-item" href="' . tep_href_link('product_info.php', 'products_id=%s') . '">%s x %s</a>'); This means tep_href_link is called by the define before executing the sprintf command, so the llink is created literally as product_info.php?products_id=%s, avoiding the URL rewriter to be able to parse it correctly. To make it work code should at least generate the real link in the module itself instead of the language file: // language file: define('MODULE_NAVBAR_SHOPPING_CART_PRODUCT', '<a class="dropdown-item" href="%s">%s x %s</a>'); // module: echo sprintf(MODULE_NAVBAR_SHOPPING_CART_PRODUCT, tep_href_link('product_info.php', 'products_id=' .$p['id'] ), $p['quantity'], $p['name']); or even more logical and simpler: // Module: echo '<a class="dropdown-item" href="' . tep_href_link("product_info.php", 'products_id=' .$p['id']) . ' ">' . sprintf (MODULE_NAVBAR_SHOPPING_CART_PRODUCT, $p['quantity'], $p['name']) . "</a>'"; // Language file: define('MODULE_NAVBAR_SHOPPING_CART_PRODUCT', '%s x %s'); I'll still have to figure out how to modify this app to parse links with {X}Y after the product id.
  15. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Hi @ecartz, I finally got a free day to check this contribution. I found your solution will not work because Usu_main has to be initialized after set_session_language and before ensure_navigation_history to work. If not, links are correctly generated but $_GET variable is not updated and Phoenix redirects to index in all pages that expects parameters, like product_info. I found its code could be placed in application.php inside the set_session_language function, or it could be loaded by creating an entry inside hooks table between those two functions (an startApplication method). But as far as I see it implies giving a non-consecutive numerical sort order to the table entry (something like _16B_start_SEO ) and it seems a bit dirty to me. Is there a cleaner way to do it? Another thing: I've seen fix_numeric_locale is being called twice (from a hook and inside set_session_language function. Is this needed or has the function call just been forgotten? Files outside ROOT seems to work now; I'm into fixing the ENABLE_SSL stuff. Is there any post where the changes introduced has been explained?
  16. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    That's correct for other SEO contribution. This is Ultimate SEO 5 and it's different, it needs a lot more tweaks. I wonder if the contribution name should be changed to avoid this kind of confusions.
  17. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    There's been deep changes that affect the way it is loaded. I also figured out a fix for files not in the main directory but I'll still have to test it deeply. I have to find some time to test it but currently I have no free time. I hope next month I'll find it.
  18. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    Thanks I'll do as soon as I have the time. I have to dig on how application_top is being processed to see where I must inject the code.
  19. piernas

    ULTIMATE Seo Urls 5 - by FWR Media

    This is not the same addon (Seo URLS 2.1 vs SEO Urls 5 Pro). I've had not much time lately but I was trying to adapt it to the latest phoenix. Its way to add code to the application is very different and I'll need more time to make it work without core code changes.
  20. piernas

    Ultimate Seo Urls 5 Pro

    I know, I've had problems with AJAX requests to /ext not being processed by the addon before. It's a complex addon and will take time for me to fix and test the issue but it's necessary for people like me with hundreds of already search engines indexed products and multilingual shops. Will try to find a better solution - if I'm able to fix it.
  21. piernas

    Ultimate Seo Urls 5 Pro

    I'm sorry I'm really busy these days. I'm not familiar with phoenix reviews - could you tell me how does it work? I cannot find the "write review" button to check how to add this to the addon. Where is it now?
  22. This is a admin sidebar that replaces the stock menu. It can be expanded or condensed on a column of icons. It also adds bootstrap framework to admin. Hope you enjoy it. Please comment here if you have any improvements or find any problem. Download: https://apps.oscommerce.com/ebjVY&admin-responsive-sidebar-with-bootstrap
  23. piernas

    Ultimate Seo Urls 5 Pro

    Fine I meant I fixed after uploading the file. Will update later.
  24. piernas

    Ultimate Seo Urls 5 Pro

    Did you see any page that should not have the tag? At first I cannot think of any...
  25. piernas

    Ultimate Seo Urls 5 Pro

    1. Language class has been fixed. USU 5 does not use $lng but its own class to handle languages. It was loading languages class file without setting $lng so the next part of code that tried to load the class triggered that error. I think it could be good to load the class in navbar core module by checking first if the file has been included already (as other modules like paypal does). What do you mean with "should be secured"? 2. I made the hook to load only if multilanguage is enabled for USU5, so no need to selective copy. 3. it's not throwing any errors here but I'll watch it . Where is cache object set? 4. I'll do on the next release.
×