Jump to content

adrian

PW-Moderators
  • Posts

    10,741
  • Joined

  • Last visited

  • Days Won

    345

Posts posted by adrian

  1. Just now, Tyssen said:

    I think the issue may be that there are hard-coded links to some pages in templates.

    So long as PagePathHistory is enabled, this shouldn't matter, but probably better to change those to get the pages by ID rather than name / url anyway.

    I don't think I am alone in my belief that page titles should match page names in almost all cases and it's too hard to expect site editors to remember to change the name when they update the title, hence my module.

  2. 7 hours ago, bernhard said:

    Great @adrian thank you!

    Would it be hard to place that Info on top of the request panel and make it uncollapsed by default? It's quite hard to find right now and I guess in many cases that information is the most important information you want to find. And it does not do any harm on regular requests as it will only be shown in redirect bars 🙂 

     

    Sure - compromise - it's still below the Page Info, but it's right below it and open by default.

    • Thanks 1
  3. Thanks for the idea @bernhard - I've added it in the latest version - you'll find a new "Redirect Info" section in the Request Info panel. It is enabled by default on new installs, but will need to be manually enabled on existing ones.

    image.png.3c785f699a1d4844510bee775bc4a6cb.png

    • Like 3
  4. The End Session button should still be a thing. Basically it will kill the user switcher session and leave you logged in as the currently selected user with no option to change.

    As for a back to xxxx user, I am with Robin - there should really be no need for that, especially if you limit the available users via one of these settings. This lets you remove frontend only users etc from the list making it much more manageable to find your own superuser account to switch back to.

    image.thumb.png.3c3485b597189b1df0c1eca633c7a4f7.png

    • Like 2
  5. Hi everyone,

    Got a new feature for you today (at the request of @eydun). 

    First some background - some of you might not even be aware that Tracy saves .html files of it's bluescreen stacktrace when an exception is thrown. The cool part about this is that these happen when Tracy is in production mode on a live site, so if you get a notification (via email or slack) that an exception was thrown, you can navigate to /site/assets/logs/tracy and load these html files in your browser and inspect the code (including values passed to functions etc). This can make debugging much easier because you don't need to try to replicate the error again.

    The new feature today makes viewing these much easier with a dedicated panel listing all available exception files. Simply click on one to have it display as it would if the exception was thrown while you were interacting with the site.

    The icon turns red when there are new exception files since your last visit and it shows which ones are new.

    The clear button will remove the bluescreen so you can return to your site without needing to reload the page.

    image.thumb.png.248c2c9fa76d75d9912750a152c376a7.png

     

    The Validator panel also had a revamp. It now uses https://validator.w3.org/nu/ because https://html5.validator.nu/ was no longer working and I have updated the CSS to support their new output DOM.

    For those of you who don't know, the key value to the Validator panel is for local dev sites where you don't have a publicly accessible URL to use - this sends the HTML of the page directly.

    image.thumb.png.1a642b4a252414a17763583de7fe7812.png

     

    Also included are some PHP 8.2 deprecation fixes.

     

    • Like 9
    • Thanks 1
  6. Hi @teppo - sorry for the delayed response.

    #1 - from what I can tell, the instances of "There is no difference between these revisions." are only an issue when saving via the API (in this case a frontend form). Not sure if it matters (seems unlikely), but it is an AJAX saved form and fields are saved with "setAndSave()" - maybe that is a factor?

    #2 - is this case, it's an admin save.

  7. Hi @teppo - firstly, just another mention of how important this module is to me - it's always one of the very first installed on every site, so thank you :)

    Now to some issues / suggestions if I may.

    1) We often find lots of revisions listed for a field that simply return "There is no difference between these revisions". I don't really understand why a revision is stored if there were no changes made. It seems like unnecessary DB bloat, but more importantly, it makes it very hard to find the revisions where changes actually are made. Do you think this behaviour could be changed so that no revision is stored?

    2) We often see changes to page reference fields (checkboxes in this case, but I expect it probably doesn't matter the inputfield type) where changes are recorded, but in reality, they weren't actually changed - why would they be recorded as being removed and then added again?
    image.png.fee8c8e19abc1b92dfee5f4588979a84.png
     

    3) When you have a page template where the width of fields is a small percentage (25%, 33%, etc), the interface for viewing the revision history is quite awkward, with lots of horizontal and vertical scrolling required. I was wondering whether you think it might be a better experience to load the revisions in a PW panel (pw-panel - https://processwire.com/blog/posts/pw-3.0.15/)

    Thanks so much for considering and let me know if there is any way I can help with any aspect of this.

    • Like 1
  8. Just a follow up to note that my issue with that "WireDataDB sourceID must be greater than 0" error was due to a hook on Pages::saveReady that was setting $page->meta even when the ID of a recently cloned page was still 0. Not something anyone else is likely to come across, but just in case, now you know to be aware of it :)

    • Like 1
  9. 1 minute ago, Roych said:

    Hmm, I was just playing with the module and I added 

    <?php namespace ProcessWire;

    in front of the module files, and it seem that it started to work ... Not sure if this was the case at all.

    R

    Interesting - I wonder if you have the file compiler turned off in your config file - perhaps the "moduleCompile" setting?

     

  10. @gebeer - are you running PHP8? If so, it might make a difference if you update to the latest Tracy - back in March there was an update to the Tracy core (which is PHP 8 only, although I maintain older versions within the package for those on older PHP versions so upgrading won't hurt).

    The other thing to make sure is that the "Use Native PHP Session" setting is unchecked.

    If those don't help, I am not sure what to do at this point - I can't count the amount of time I have spent workaround issues with SessionHandlerDB and Tracy.

     

  11. Ah, thank you - I did a quick scan for function_exists checks around all occurrences of posix_getpwuid but missed that one because of the check for posix_geteuid - my eyes didn't see the difference.

    Fixed in the latest version.

    PS - yes, feel free to add new threads for bug reports - would definitely be preferable.

×
×
  • Create New...