- Issue created by @DamienMcKenna
- last update
11 months ago run-tests.sh exception - last update
11 months ago run-tests.sh exception - last update
11 months ago run-tests.sh exception - 🇺🇸United States DamienMcKenna NH, USA
I've committed the initial changes, but additional work is needed.
-
DamienMcKenna →
committed 03646ee3 on 7.x-3.x
Issue #3444839 by DamienMcKenna: Coding standards fixes.
-
DamienMcKenna →
committed 03646ee3 on 7.x-3.x
- Issue was unassigned.
- Status changed to Needs work
11 months ago 11:32am 3 May 2024 - 🇭🇺Hungary nagy.balint
@DamienMcKenna
Hi!
There is an issue with the previous commit, as there is an extra comma which causes:
ParseError: syntax error, unexpected ')', expecting variable (T_VARIABLE) in drupal_load() (line 2620 of views.module)Attached patch.
- 🇭🇺Hungary nagy.balint
The commit in #4 also broke ajax in the views admin, since TRUE and FALSE are not valid in javascript.
Seems to be several issues with this commit :(
- Status changed to Needs review
9 months ago 4:31pm 16 July 2024 - 🇭🇺Hungary nagy.balint
I put it on needs review, since the dev release in the current state is not really useable...
-
DamienMcKenna →
committed 651f6938 on 7.x-3.x
Issue #3444839 by nagy.balint: Coding standards fixes.
-
DamienMcKenna →
committed 651f6938 on 7.x-3.x
- 🇺🇸United States DamienMcKenna NH, USA
Thank you for the fixes, I've committed patch #8.
- Status changed to Fixed
25 days ago 10:03am 13 March 2025 Automatically closed - issue fixed for 2 weeks with no activity.