Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

piernas

Members
  • Posts

    700
  • Joined

  • Last visited

  • Days Won

    17

Everything posted by piernas

  1. @marokech please download latest from here: https://apps.oscommerce.com/Imrlq&ultimate-seo-urls-rewritter-for-phoenix and please post any problem here:
  2. I'll try to update the ones I've made in the past while I have some free time
  3. 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.
  4. Updated for Phoenix: https://apps.oscommerce.com/2iSYY&editable-shipping-table-phoenix
  5. Yes you'll just have to add it to the $alternate variable.
  6. Hi David, I dodn't know about that tag - I'll add it in the future once the app has been tested.
  7. 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
  8. 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;
  9. @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
  10. Thank you, I'll take a look at those errors. Does the app work?
  11. 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
  12. 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.
  13. 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?
  14. 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.
  15. 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.
  16. 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.
  17. 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.
  18. 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.
  19. 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?
  20. Fine I meant I fixed after uploading the file. Will update later.
  21. Did you see any page that should not have the tag? At first I cannot think of any...
  22. 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.
  23. move HT module to Hook -> good idea change tpl_ file -> I have a problem because templates uses a $usu5_multi class not declared as global in the modules. Any ideas?
  24. It's ready. - Latest oscommerce (phoenix 1.0.3.0) compatible. - Fixed more cache issues. Now sqlite and memcache are only enabled if system supports the methods. - Moved admin edits to hooks. Now only two files on catalog needs to be modified. - Removed database table constants. - TODO: Allow to processs links outside root folder (ie. ext/directory). This is useful for addons like Twitter typeahead. Can someone test the atached files before uploading to addons? USU5_210.zip
×
×
  • Create New...