Jump to content

adrian

PW-Moderators
  • Posts

    10,741
  • Joined

  • Last visited

  • Days Won

    345

Posts posted by adrian

  1. Really great writeup @artfulrobot

    5 hours ago, artfulrobot said:

    Logging is weird and non-standard. I prefer one chronological log of well categorised events, but instead we have lots of separate files. This is a bit weird.

    You might find that TracyDebugger helps with this to some degree - its "Processwire Logs" panel combines all logs into one ordered reverse chronologically. It also highlights when you have new errors / notices since you last viewed the site.

    • Like 3
  2. @Abe Cube - I know this is very old, but perhaps it will help someone else. I just had this same error and it turns out is was due to a mismatched class name and file name. I was testing something and my file was a typo: ProcessHiddenPageText.module.php while the class name was the correct ProcessHiddenPageTest.module.php

    As soon as I fixed the filename and did a modules refresh, everything worked as expected.

  3. 7 hours ago, ottogal said:

    Just updating the TracyDebugger module did not affect the files and directories in the panel/ subdirectory

    Hi @ottogal - not sure why that would happen. How did you update Tracy? I suppose you can't really tell anymore, but I wonder if the permissions for that folder were somehow preventing it from being writable by PHP. Anyway, updating is not something individual PW modules have any control over so certainly not an issue with Tracy per se. I shouldn't be related to the PHP version, but you do really need to get that updated - it was EOL'd 5 years ago so you've had no security updates in that time - I would think your host should be more worried about that than an old version of Adminer that was restricted to superuser access anyway - all module files in PW are restricted by htaccess.

    2 hours ago, Denis Schultz said:

    Any plans to switch to the maintained fork: https://github.com/adminerevo/adminerevo ?

    Thanks for the heads up @Denis Schultz - I hadn't seen that, but I'll definitely take a look and see what it would take to switch.

  4. 58 minutes ago, gRegor said:

    @adrian Oh interesting, I had just updated it before posting, but I see there's actually 4.25.18. PW might have had .17 cached as the newest version. I'll update again.

    Nope, 18 was a brand new version (after your bug report).

    39 minutes ago, gRegor said:

    4.25.18 appears to still have the issue. It's in these lines:

    Sorry, I missed that line - I did get the one (and one other) from your initial report, but that was one I missed.

    The latest should hopefully be OK now.

    • Like 1
  5. 1 minute ago, ryangorley said:

    I'll add, perhaps for @ryan to consider, that an equally good solution would be for ProcessWire to borrow a behavior from elsewhere. In WordPress if I want to reset a slug I just clear the field and save the post. It then auto-populates that field from the post title as it does when the post is first creation. In ProcessWire clearing the slug field and saving the page just reverts the fields to their prior content. A small, but nice feature in WordPress. :)

    You might find this useful: https://processwire.com/modules/page-rename-options/

    • Like 2
  6. If you're getting that message after clearing the flag, then it means there are more errors being logged. The whole idea is so you don't get multiple emails each time an error is logged, so once you fix the code that is causing the error, you clear the flag and it won't appear again until another error is logged and emailed to you. Make sense?

  7. @tires - an addition to the regex on this line (https://github.com/adrianbj/GetVideoThumbs/blob/31a7fb443abd1b1cb9143840954f3f25d52d4fe5/ProcessGetVideoThumbs.module.php#L129) should be able to support it. If you want to give it a go and send me a PR, I'd be happy to include it.

    PS - that said, most of the videos on piped don't seem to play for me - some do, but it's pretty hit and miss, so maybe not a very stable service at the moment.

    • Like 1
×
×
  • Create New...