adrian

Sorry I didn't catch that ? So for some reason your install is behaving differently. I think I have narrowed it down to multibyte support or iconv. Could you please run these in the console and let me know what you get: d(function_exists("mb_internal_encoding")); d(function_exists("\\iconv"));

@PWaddict - thanks for checking that code in the Console, but it looks the same as what I am seeing, so I can't figure out why you're getting a different result returned from the module. Not sure where to go from here other than asking you to test on another PW install.

This is interesting: Are you seeing the first, rather than the second result? See the last commit where those booleans are passed to the cleanBasename() method: https://github.com/adrianbj/CustomUploadNames/commit/9428a18c4cf67a40c7ef6bd6b63ae600a28086e3 Could you please check that the version you are using has those lines and also please test that code in the Tracy console: $pageFiles = new Pagefiles($page); d($pageFiles->cleanBasename('"Test ä".png')); d($pageFiles->cleanBasename('"Test ä".png', false, true, true));

Weird, that filename format patter for me returns: test_a_1545255464.jpg which should be exactly what we expect - correct?

You can use: $input->post('reset-button') or $input->post->{'reset-button'} This is not a PW issue, but rather a PHP one - you just can't use hyphens like that. That is why PW field names use underscores and not hyphens.

@PWaddict - I know you'd like me to just revert to that previous version but I would like to figure out why it seems to be working ok for me and not you. Can you please post your settings for the rename rule and an example of a field value that is causing htmlentities to be replaced with underscores.

Go to Modules > Core > AdminThemeUiKit. You can choose the Gravatar option or you can edit the user template to add an images field which can be used for manually added images. Remember that the user template is a system template so you need to go to Setup > Templates > Filters > Show System Templates.

Actually, I messed up that reversal, but I decided to investigate the issue you're having with 1.2.3 and I can't reproduce. This is what happens for me in a repeater - note that the ä is replaced with an "a" as expected. I don't think it should matter, but what version of PW are you running?

Thanks for noticing that - I have reverted that change and bumped the version number to 1.2.4 to make sure anyone who upgraded to 1.2.3 sees that there has been an update.

@PWaddict - I just committed another update which makes use of PW's core cleanBasename method for generating the final filename. I think this should be an improvement in certain situations. Please let me know if you have any problems with it.

Yep, good catch - please try the latest version.

Hey @tpr - I've started seeing something weird in the page tree with hidden pages. They don't show until I mouse over the tree itself: Let me know if you can't reproduce and I'll try to narrow it down for you.

You can also switch PW versions instantly with Tracy. By default it works with versions which are archived by Ryan's PW Upgrades module, but there should be nothing stopping you from recreating those archived version manually by creating .wire-x.x.x folders with the contents of that version's wire folder.

No problem! BTW, I have updated the module so it is now tagged as requiring php 5.4 so hopefully no-one else comes across this error.

Well if you upgrade you won't need to change anything. But in the meantime look at line 106 (where that error is coming from) and you'll see $this->wire('languages') - change that to wire('languages'). Do the same for other instances like $this->wire('session') - change this to wire('session'). Does that make sense?

That's because Tracy requires PHP 5.4. 'requires' => 'ProcessWire>=2.7.2, PHP>=5.4.4', This is a requirement of the Tracy core and not specifically my module. Please note that PHP 5.4 has been out for 6 years now. 7 has been out for 3 years. If your host won't let you upgrade to 7 I would look elsewhere!

PHP 5.3 is the issue - it doesn't let you use $this in anonymous functions. I had forgotten about this as I haven't used 5.3 in years. The quick fix for you is to change those instances of "$this->wire" to just "wire" I think probably many of my modules will be affected by this ?

While I am waiting to hear back, if you have a critical error like this, you can remove a module by deleting it's files from /site/modules and it's entry in the modules database table which is easy to do with Adminer which is packaged with Tracy.

Hi @Tobi - sorry you're having problems. What version of PW, this module, and PHP are you running?

__() works for me in a config.php file here: https://github.com/adrianbj/CookieManagementBanner/blob/master/CookieManagementBanner.config.php I know that's not very helpful though ? Perhaps it's a namespace issue - are you declaring the ProcessWire namespace, or have you turned off the file compiler (which is needed if you don't declare it).

Does modsec baulk at [iframe] ? If it works you could write a simple textformatter that translates that to <iframe>

That module converts the url to an iframe link: https://github.com/teppokoivula/TextformatterGoogleMaps/blob/master/TextformatterGoogleMaps.module#L77 Do you think you could modify that line to change it to the parameters you need?

How about using this instead: http://modules.processwire.com/modules/textformatter-google-maps/ Much better than pasting in iframe code into an RTE ?

I'd also love to see more/better debugInfo for other PW objects as well, as per: https://github.com/processwire/processwire-issues/issues/575

Or change it in the database - via the link to Adminer in the RequestInfo panel in Tracy is a nice shortcut, but however you get there is fine.