Jump to content

adrian

PW-Moderators
  • Posts

    11,263
  • Joined

  • Last visited

  • Days Won

    374

Everything posted by adrian

  1. A couple of small updates to the Adminer panel / process module. 1) It is now restricted to the database for the PW install. Previously you could access all databases available with to the PW DB user. If some of you out there don't like this change, let me know and I can make it optional, but I think it's a better default (cleaner and a little safer). 2) It's now possible to uninstall the Adminer process module and it won't be automatically re-installed each time you upgrade Tracy - @Rudy was concerned about this with his setups. Obviously if a user has superuser access they can still install if they want, so it's more a case of removing something tempting for users without sufficient skills, rather than being a true change to security. On this note - Adminer is restricted to superusers (same with the Console, Snippet Runner and File Editor panels).
  2. Hi Robin, I don't actually ever use the "_items" suffix myself - I always go with "cars" and "car" for the parent and child templates, respectively. That approach was to make things as logical as possible without the user having to change these settings. I am definitely keen to take a different approach though and the pluralize library seems like a good option. I see that it contains lots of rules for non-standard "s" suffixes, although it doesn't look like there any non-English ones provided. The other concern I have is the "Singular words with no plurals" list (https://github.com/blakeembrey/pluralize/blob/298eb67f19369dd30207c0817eda4f0045c6c1a7/pluralize.js#L393) - won't these result in the parent and child templates having the same name? Do we just need a check to make sure they're not the same and if so empty the child temptate name field and prevent the form from submitting until it is completed? Any thoughts?
  3. Great - thanks for testing. I just committed another minor update that renames and reorders some of the settings and divides them into "required" and "optional" to make things a little more logical and quicker to complete.
  4. Is this what you are looking for? https://processwire.com/blog/posts/processwire-core-updates-2.5.14/#multiple-copies-of-the-same-module This is what @arjen is referring to above. That said, I still think a hook approach is better if you can.
  5. Done - let me know how it goes for you. I also fixed some other PHP7 issues with the Option 2 approach.
  6. Hi @Robin S - I have fixed the bug regarding child page creation - I think it was a change to "empty" in PHP 7 that was the cause. I'm in transit now, but I can take a look at the new option for being able to set the parent page title in the next few days.
  7. Actually please try a couple of versions back also. I'll certainly get this fixed when I'm back on the 12th, but it would be good to have things working for you before then. Also if you want to submit a PR for your feature request above is be happy to accept it. Perhaps discuss a little with @Jonathan Lahijani because I know he is planning some new features also.
  8. @Robin S - would you mind checking the previous version? Not sure whether something new in this module might have broken it or possibly a change in PW.
  9. Thanks @Robin S PR merged and modules directory updated.
  10. @bernhard - there is a name () method from Adminer which I think will handle this. Only catch might be not wanting it in the panel version?
  11. @Robin S - I wonder if a simple temporary solution might be to turn off error reporting at the top of the Adminer process module file?
  12. Thanks for the PR. Maybe we need to get those plugins cleaned up. I'll take a look when I can unless you beat me to it☺
  13. PS - I'll take a look at the possibility of custom compiling when I'm back. Also do you have any ideas on how to get around the debug mode issue?
  14. Thanks everyone for the debugging. Would someone (maybe @Robin S) mind putting together a PR with his suggested fix for the &db issue and if you wouldn't mind also @matjazp's request for a custom port. I can easily merge a PR on my phone but not keen on coding ☺
  15. I'm on the road now so can't help much but interesting @bernhard isn't having issues. Also weird that the file is referencing itself regarding the headers sent stuff. That said I can't see it being a core Adminer issue either. Hopefully you guys can help to figure it out for me ?
  16. @kongondo and @Robin S - can you please try the attached version to see if that fixes those errors on Windows please? ProcessTracyAdminer.module
  17. I have this fixed in the Adminer panel version, but struggling to get it sorted in the Setup > Adminer version. Not sure I'll get this sorted tonight before I leave I'm afraid. I'll just leave it this way for now then - will tweak when I am back if needed.
  18. Done, although just realized I should tweak this so that the PW menu system is retained - coming soon ? I'll look into this shortly.
  19. Quick note to let you know that it's also possible to run Adminer directly via: /processwire/setup/adminer/ which may be preferable if you have lots of work to do and want it open in a more "permanent" way. At the moment I set the page as hidden so it actually won't appear under the Setup menu - I can change this, or perhaps I can have a link from the Adminer Tracy panel to open it this way. Any thoughts on which would be best?
  20. Just released the new Tracy Adminer panel: https://processwire.com/talk/topic/12208-tracy-debugger/?do=findComment&comment=175420
  21. New "Adminer" database GUI panel just added. A huge thanks to @Robin S for making me take another look at this long forgotten tool and for helping with many suggestions regarding the implementation in Tracy and debugging several issues. There is one outstanding one on his local Windows dev setup that I can't reproduce at all and he also isn't seeing on a live server. He is getting session_start / headers already sent error. Could someone else on Windows (maybe @bernhard) take a look and let me know if you have the same problem as well. Actually Robin - any chance you are running the SessionHandlerDB modules on that local setup? The color scheme is configurable in the Tracy settings. I have used this already on a live server to help debug something and it would have taken much longer to go find the cpanel credentials, and navigate my way to PHPMyAdmin. This logs you in automatically and is virtually instant. I am away for the next 10 days so if you find any problems, they will most likely have to wait till I'm back. Hope you find it useful!
  22. Here's a teaser - no prizes for guessing what the panel wrapping the interface is from ?
  23. Good point ?
  24. There is a hacky way around this. This comes direct from the ready.php file from one of my sites - in this case it's for the autocomplete inputfield, but maybe something like this would be useful for you also? // hack to overcome this: https://github.com/processwire/processwire-issues/issues/550 $this->addHookBefore('Pages::find', function(HookEvent $event) { $selector = $event->arguments(0); if(is_string($selector) && strpos($selector, 'template=user') !== false && strpos($selector, 'name|first_name|last_name%=') !== false) { $selector .= ', check_access=0'; } $event->arguments(0, $selector); });
  25. I agree the current approach isn't great, but is it possible to simply add "/site/assets/MyModule" to the ignore list. Doesn't that achieve what you want? If not, I'd certainly be happy for a regex approach.
×
×
  • Create New...