Jump to content

Welcome, Guest!

By registering with us, you'll be able to discuss, share and private message with other members of our community.

Traumflug

Administrators
  • Content Count

    1,319
  • Joined

  • Last visited

  • Days Won

    35

Everything posted by Traumflug

  1. Let's not forget this: https://github.com/thirtybees/thirtybees/issues/1077
  2. Regarding improperly uninstalled modules: residuals in the database are usually harmless. More crucial are not uninstalled overrides, which one can find and uninstall with this module: https://github.com/thirtybees/overridecheck/releases
  3. Got it. This small change to Core Updater fixes it: https://github.com/thirtybees/coreupdater/commit/56ef3e2e1eb0e7ab9175477d9365ae33e6167a4e
  4. The workaround is to shorten the browser window. If I had code to fix this issue, I had committed it already 🙂
  5. A '@' is mandatory part of an email address, obviously. Perhaps you could give us a few examples of addresses which should work, but don't.
  6. This is (kind of a) syntax error message and usually easy to solve. Surprise here is, why was this message found now, while dozens of shops use this module already?
  7. Uhm, nice code, but it solves a not existing problem. Code for properly handling long menus is already there. ... except for browser windows of a certain height. Making the window a bit shorter puts submenus not under, but to the right of the parent menu item on hovering, keeping everything available: Rather than adding yet another interface item to this already pretty complex menu handling code I'd much prefer to fix this length calculation problem. Didn't come around to do it myself, yet, as the workaround is so easy.
  8. I just tried, the server appears to work fine. Could you simply try again? Maybe you've hit a weak spot. Updating multiple times, even after failures, should work fine and give the expected results. If it still doesn't work I'd need access to the server to find out why these packages arrive empty.
  9. This is the fix: https://github.com/thirtybees/niara/pull/17/commits/0d91e2cc3a585cbd998c9b2caf295a6a6f3d5fef The translation is there, just well hidden. The fix moves it into the right place.
  10. It's better to change it to just "home". Then this module works for other themes as well.
  11. Did you try to edit php.ini, like I described in an email to you? No idea which kind of "security issue" they have in mind, it's just a shortcut for more complex web requests. One can easily write code doing the same without this feature enabled, and most of thirty bees' code already does. thirty bees 1.0.7 doesn't test for it on installation, still uses it in a few places.
  12. File manager isn't a back office controller, so it can't get put into a menu item. However, wherever one can edit page content, the editor for this field has a button to open this file manager.
  13. Obsolete files are usually harmless. Only a few exceptions, like template overrides, change behavior. The file you found is apparently a log file generated by some non-thirty-bees code, so it'd be better to look into it, act on the error reported as necessary, then to delete it.
  14. If it's a developer installation, one usually has to deal with web server and you running as distinct users. Which means, files created by them also get distinct privileges. One can deal with this automatically using extended access control lists (ACL): http://jump-ing.com/traumflugs/github_best_practices#file_permissions
  15. Opt-Out ist weiterhin in Ordnung. In diesem Verfahren ging es um ein Cookie-Banner, bei dem der Haken bereits gesetzt ist und das Banner so quasi zu einem Opt-In macht. Denn man muss den Haken aktiv entfernen, um Cookies zu vermeiden.
  16. See themes/niara/sass/partials/_footer.scss, change it, then rebuild the theme. If you don't want to rebuild the theme, see #footer in themes/niara/css/global.css
  17. There were quite a few missing 😳 https://github.com/thirtybees/thirtybees/commit/3ca6967dc8c44badd7e1d8b455320f6660a87b91
  18. Got it. @veganline kindly gave me access to the server. The TLSv1.2 message is a red herring. Test fails, but it's an optional test, which doesn't prohibit installation. Actual issue was a missing error message. The test for ensuring PHP's fopen() takes URLs failed, but didn't report an error, because there is no matching error message. Setting allow_url_fopen in php.ini to 1 should allow installation. And I'll add the missing error message to the sources, of course.
  19. Without looking it up, I'd guess it indeed renames it from 'admin' to something else. This happens right after installation and the user gets notified about this. No idea why people would want to rename this folder to something less safe, like 'admin'. Just don't do it. Renaming it to something not starting with 'admin' breaks a shop installation as well. Don't do this either. It's an e-commerce software, not a funny-user-idea-healing toy 🙂
  20. Traumflug

    Broken images

    P.S.: I'd also download a few images and open them in an image viewer to find out whether they're actually JPEGs (or PNGs with suffix .jpg). I've seen shop installations where properly named files were there, but none of them being actual images.
  21. Traumflug

    Broken images

    Looking at this page, one image is missing: https://product.solutions.org.nz/remote-car-alarm-with-central-locking-and-engine-disable Digging a bit deeper, this is the image URL: https://product.solutions.org.nz/194-community-theme-default_cart_default/remote-car-alarm-with-central-locking-and-engine-disable.jpg This URL looks fine to me. Shop is apparently using theme community-theme-default, this theme has the old image type names (with _default), so that's what's expected. Also, some other images work, which makes a glitch in image finding unlikely. Broken code usually applies to all images, not just a few. I'd start looking at which images are actually on disk. This image should be in img/p/1/9/4/ and next to index.php and the original file, 194.jpg, there should be one additional file for each image type. Similar to this (this is for image no. 21): 21-community-theme-default_cart_default.jpg 21-community-theme-default_home_default.jpg 21-community-theme-default_home_default_smaller.jpg 21-community-theme-default_home_default_smallest.jpg 21-community-theme-default_large_default.jpg 21-community-theme-default_medium_default.jpg 21-community-theme-default_small_default.jpg 21-community-theme-default_thickbox_default.jpg 21.jpg 21-Niara_cart.jpg 21-Niara_home.jpg 21-Niara_home_smaller.jpg 21-Niara_home_smallest.jpg 21-Niara_large.jpg 21-Niara_medium.jpg 21-Niara_small.jpg 21-Niara_thickbox.jpg index.php
  22. Traumflug

    Broken images

    Well, it fixed the problem I found in my development installation. Looks like there's more than one problem involved here. Or I fixed something else, not the issue here.
  23. Excellent analysis! This timestamp was introduced here: https://github.com/PrestaShop/PrestaShop/pull/4267/commits/1e1932559758ce989a8d42d931fd650043234d21 to fix this bug: http://forge.prestashop.com/browse/PSCSX-7050 Such a timestamp doesn't make sense to me. Each image has an individual number, there should be no caching issue (unless one changes the image on the command line or by FTP). I've reverted this commit, which results in loading the right picture. On Bleeding Edge -> 1.1.x in ~10 minutes.
  24. That's actually by design and was removed intentionally: https://github.com/thirtybees/niara/commit/5991643b278f4f0343dc7e90957984013a6db98f Merchants usually want visitors to buy something, so having a delete button looking almost like a close-dialog button isn't too helpful.
  25. This sounds like a good idea. Along with all the usages of realpath() @musicmaster mentioned. Catching issues at the root instead of dealing with them on each instance. Certainly quite some work, but not too hard to do.
×
×
  • Create New...