Jim Graham

Results 203 comments of Jim Graham

@benni1516 - Just curious (and not making any assumptions); have you pulled this down and tested this or are you commenting based solely upon looking at the code changes? I...

@benni1516 - No apologies necessary ... just wanted to encourage you test ;-) All - FYI, changed status as a little more needs to be done with the regex. I'm...

What I'm trying to do is, taking into account that you _can_ traverse (and dynamically create) directories moving downward, keep the resulting sanitized path as clean as possible (and non-helpful...

@JoshuaLuckers - Thanks for beginning to take a look at this! I know it's relatively big, but in the past I've gotten caught between "it's covering too much" and "why...

I have a PR that addresses this globally (#16225) that is ready for merging but please take note of [my comment](https://github.com/modxcms/revolution/issues/6703#issuecomment-1182473333) re autocomplete being difficult to impossible to control with...

Ok, yes, I've seen this issue myself on a couple of sites where migxdb is used. It's not something to address here in this PR. It may well be a...

Actually, we should be able to create a dropdown using the data already available in the Regions tab (only touching the JS, no processors).

There's just one problem with this: although it typically is, `localhost` does not have to be assigned to 127.0.0.1; it can be assigned to other addresses within that block (1-255).

The only other thing I'm wondering is what true advantage of this utility is. I'm not saying I know there are _not_ advantages, I'm just not readily seeing them and...

> Personally I find this pretty annoying and makes the code less readable I agree with you on that when each variable is _not_ on its own line. (I wasn't...