- Issue created by @DamienMcKenna
- Status changed to Needs review
11 months ago 7:31pm 20 July 2023 - πΊπΈUnited States DamienMcKenna NH, USA
This adds a new setting that controls whether redirects should happen if the paths value is empty, with it defaulting to TRUE to match current settings.
- last update
11 months ago 82 pass, 22 fail The last submitted patch, 3: cas-n3375946-2.patch, failed testing. View results β
- last update
11 months ago 82 pass, 22 fail - πΊπΈUnited States DamienMcKenna NH, USA
Forgot the ::submitForm() changes again. Also added a description explaining when it would be useful.
The last submitted patch, 5: cas-n3375946-5.patch, failed testing. View results β
- Status changed to Needs work
11 months ago 8:04pm 20 July 2023 - πΊπΈUnited States DamienMcKenna NH, USA
I swear I've done this before X-)
This fixes the form field as it now looks for the correct variable.
- Status changed to Needs review
11 months ago 8:05pm 20 July 2023 - last update
11 months ago 82 pass, 22 fail The last submitted patch, 7: cas-n3375946-6.patch, failed testing. View results β
- last update
11 months ago 82 pass, 22 fail - πΊπΈUnited States DamienMcKenna NH, USA
There was a bug in how this worked, it was verifying that $this->forcedLoginPaths was not empty, but it actually needed to make sure that $this->forcedLoginPaths['pages'] was not empty.
This should work now.
The last submitted patch, 10: cas-n3375946-10.patch, failed testing. View results β
- last update
11 months ago 100 pass - πΊπΈUnited States DamienMcKenna NH, USA
Hopefully this will fix the last issue. (I now have the tests running locally, had previously forgotten to download cas_mock_server so the tests had been failing)