Jump to content

matjazp

Members
  • Posts

    687
  • Joined

  • Last visited

  • Days Won

    4

Everything posted by matjazp

  1. The problem is in WireFileTools.php in rmdir() method in foreach loop. The $path coming into rmdir() contains ending slash, and $pathame (actual directory name) eventually contains two slashes , that makes allowPath() method to throw an WireException. The possible solution is simple: $path = rtrim($path, "/");
  2. Yep, that's true. How about making this as an option? Most of the time work on templates is done on wide screen. Also, some might prefer to have fieldsets on the main screen, not on separate tabs, so I would make this configurable too. Just an idea, not a feature request.
  3. Cool! Add an arrows-h icon to the InputfieldMarkup? Make the widths with the slider instead of text input? Dynamically show the width of fields as they change?
  4. Thanks for the module. Maybe this could be in the core... Any chance of supporting AdminThemeDefault? If not, then you could possibly check if the user has Uikit theme active before injecting css file and applying classes?
  5. https://github.com/processwire/processwire-issues/issues/704
  6. I upgraded PW to the latest dev using the upgrade module from Ryan (I normally upgrade by hand) and got the message that Upgrade files are already present. Please remove them before continuing. Clicking on Remove button didn't help, had to manually delete /site/assets/cache/ProcessWireUpgrade folder. I'm on windows, I think this is important. Anyone else?
  7. Yep, it's me ? I had an error (premature end) in my 404 template file.
  8. I tried new redirect option and manually created redirect, but instead of a redirect, I get 404 page. I'm probably doing something wrong ...
  9. @adrian when you get back, please take into account $config->dbPort setting. I'm using non-standard port and this should be honored in ProcessTracyAdminer.module something like that: $port = wire('config')->dbPort ? ':'.wire('config')->dbPort : ''; ... new AdminerProcessWireLogin(wire('config')->dbHost.$port, wire('config')->dbUser, wire('config')->dbPass, wire('config')->dbName),
  10. I don't know if Ryan need to know that. It seems there are no (known?) side effects with this paths, maybe there could be an issue with adminTemplates if PW is running in subdirectory? Ryan is busy, don't won't to bother him. And there are other issues that need his attention...
  11. Thanks for this, very cool! When you toggle all and then try to collapse one object/class by clicking on the down arrow, the first click doesn't collapse, had to click it again. Nothing to do with Tracy, but I noticed that some paths in $config are different, like $config->paths: ProcessWire\Paths #3457 _root protected => "C:/inetpub/wwwroot/" (19) data protected => array (206) wire => "wire/" (5) site => "site/" (5) modules => "wire/modules/" (13) ... fieldTemplates => "site/templates/fields/" (22) adminTemplates => "C:/inetpub/wwwroot/wire/modules/AdminTheme/AdminThemeDefault/" (61) sessions => "C:/inetpub/wwwroot/site/assets/sessions/" (40) and $config->urls ProcessWire\Paths #cc50 _root protected => "/" data protected => array (206) wire => "wire/" (5) site => "site/" (5) modules => "wire/modules/" (13) ... adminTemplates => "/wire/modules/AdminTheme/AdminThemeDefault/" (43) AdminThemeDefault => "wire/modules/AdminTheme/AdminThemeDefault/" (42) (adminTemplates has the slash at the beginning). Is this how it should be?
  12. @BitPoet thanks. I'm sure we will not add an exception for processwire.com (or any other domain). I'm not sure Ryan is aware of this, as it looks like (looking at the SPF record) he was using mandril before. I hope this post will reach him somehow ...
  13. Hi! Today I changed my email address associated with my PW forum account. PW sent an email to my new email but unfortunately, I didn't get it. Luckily I could change my email again to g*ail address and this time I received an email instantly. The from address is: "ProcessWire Support Forums ryan@processwire.com via sendgrid.net". G00gle explains: You'll see "via" and a website name next to the sender's name if: The domain it was sent from doesn't match the domain in the "From:" address. For example, you got an email from john.smith@gmail.com, but it could've been sent through a social networking site and not Gmail. The email was sent to a Google Group from a domain that has a "p=reject or p=quarantine" DMARC policy. You can't remove the "via" next to someone's name. Gmail shows this information so you're aware of where your messages are coming from. It looks like our (my company's) SMTP server doesn't accept "those" emails. Is there something that could or should be done in that regard? G00gle says this: If you send messages using a bulk mailing vendor or a third party affiliates, check out Gmail's Bulk Senders Guidelines, including: Publish an SPF record that includes the IPs of the vendor or affiliates which send your messages. Sign your messages with a DKIM signature that is associated with your domain. Make sure the domain in the "From:" address matches the domain you're using to authenticate your emails.
  14. Looks ok, icons are there too. Thanks.
  15. It's not applied because AdminThemeDefault class is set on body, not on html. This is what I have now and it's working here: .Inputfields .InputfieldMarkup .InputfieldContent { margin-top: 0; } div#links { margin-left: calc(100% - 250px); text-align: right; position: relative; top: 35px; z-index: 103; } .AdminThemeDefault div#links { top: .5em; } #AdminActionsList { background: none; } .AdminThemeDefault #AdminActionsList { margin-top: 1em; } Mind z-index on div#links. Because there is no $info['icon'], see the php notice in my second screenshot: array (6) title => "Copy Content to Other Field" (27) description => "This action copies the content from one field to another field on all pages that use the selected template." (107) notes => "This can be useful if you decide you need to split one field into two to allow different settings on different templates. It also makes it easy to move content from one field type to another one that is incompatible." (216) author => "Adrian Jones" (12) authorLinks => array (3) roles => array (1) in ...\ProcessAdminActions\ProcessAdminActions.module:388
  16. It's not a cache. I checked css file, no other site css file is loaded, just from wire and from the module itself. For the Core tab, it's .WireTabs top position in main-classic.css, and for the link it's div#links top position in ProcessAdminActions.css. I'm testing in Chrome. It's looking good in uikit and reno theme.
  17. Hi Adrian, I just noticed a bit odd layout of core tab and actions list link:
  18. Just to add my experience: on the installation, I also got the warning that InputfieldStreetAddress requires JquerySelectize. When installing JquerySelectize, I got: PHP Notice: Undefined index: forceLoad in ...\modules\JquerySelectize\JquerySelectize.module:127 This is probably a problem with JquerySelectize, but the error went away after a few submits (or maybe modules refresh?) so I didn't bother with that.
  19. It also looks that $files->rmdir() method (in WireFileTools.php) could/should check if $path is provided with trailing slash or not. When this method iterates over the directory in recursive mode, the $pathname contains two slashes. But it still manages to delete the folder, php takes care of that...
  20. This error message: Unable to remove: C:/inetpub/wwwroot/site/assets/cache/WireTempDir/.ProcessModuleInstall/ is issued in WireTempDir.php in line 290 when calling $this->rmdir() method in the WireTempDir.php when an module is installed or updated: protected function rmdir($dir, $recursive = false) { $dir = rtrim($dir, "/\\") . DIRECTORY_SEPARATOR; if(!is_file($dir . self::hiddenFileName)) return false; unlink($dir . self::hiddenFileName); return $this->wire('files')->rmdir($dir, $recursive); } When I commented if(!is_file($dir . self::hiddenFileName)) return false; all those temp files starting with dot got deleted. And .ProcessModuleInstall folder along with the WireTempDir also got deleted. If I again uncomment above line and the install a module, WireTempDir is created, the file .wtd is there and .ProcessModuleInstall folder is created and the error is again present. I'm not sure if this is intended behavior? Also, $dir contains forward slashes so $dir = rtrim($dir, "/\\") . DIRECTORY_SEPARATOR; should be $dir = rtrim($dir, "/\\") . "/"; ?
  21. Upgraded from1.0.3 to 1.0.4, the content of the formats_overrides.php is preserved. Pages: Successfully copied C:/inetpub/wwwroot/site/modules/.FieldtypeStreetAddress/formats_overrides.php to C:\inetpub\wwwroot\site\modules\FieldtypeStreetAddress/formats_overrides.php
  22. @pwired thx for response. Using latest dev PW, Win 8.1 Pro with IIS, logged as local admin, have the rights to this folder, the owner is IUSR (user running web server on IIS), permissions looks ok. I don't have access to my home computer right now, but will report back later with the update. Maybe there is a problem deleting a file/folder starting with dot on windows?
  23. There is /site/assets/WireTempDir folder. There is a bunch of empty folders with random names, starting with a dot, eg. .PFM0O90222400 1521174131OsICtIiGFJdM What is the purpose of those folders? Can they be deleted? Also, there is a folder .ProcessModuleInstall and a message in /site/assets/logs/wire-temp-dir.txt: 2018-09-14 05:02:39 admin http://localhost/processwire/module/download/ Unable to remove: C:/inetpub/wwwroot/site/assets/cache/WireTempDir/.ProcessModuleInstall/ I guess it is related to windows environment.
  24. Hi, Steve. Thx for the module. Is there a way to prevent adding the country code to the zip? eg. from "SI- 2000 Maribor" to "2000 Maribor"? Here are some examples of correct addressing in Slovenia: Maja Čebelica Vrtna ulica 1 5000 Nova Gorica Gospa Milena Galeb Morska cesta 11 a 6000 Koper Stanovanje 14/II Gospa Nada Skok Cvetlična ulica 8 2270 Ormož Prodajalna AVTO p. p. 1635 1001 Ljubljana Pošta Slovenije d.o.o 2500 Maribor Firma AG Hauptstrasse 28 CH–5400 Baden Schweiz As seen from examples, street address is not mandatory. Stupid question: what's the benefit of using this field instead of plain textarea?
×
×
  • Create New...