post in admin page of addon does not work since commit 30140589c

Mon, 22 Aug 2022 08:48:31 -0700 from streams

@mike

I pulled the dev today.

The admin page of my addon shows but the post does not work anymore (submit button).

I went back to previous commits and found that commit 3623e8a01 works and the next one 30140589c does not work.

A difference between the too is line 27 in Addon.php

$func($a);

versus

$func();

Is this a bug or a feature?

Giac El Vecio

Mon, 22 Aug 2022 09:24:08 -0700 from Diversi Spiritus

is Addons.php ;)

Giac El Vecio

Mon, 22 Aug 2022 09:46:59 -0700 from Diversi Spiritus

is the line 173 in
Code/Module/Admin/Site.php ok?

"set_config('system', 'force_queue_threshold', $force_queue);"

Giac El Vecio

Mon, 22 Aug 2022 09:49:33 -0700 from Diversi Spiritus

shouldn't it be
"set_config('system', 'force_queue_threshold', $force_queue_threshold);"
?

Mon, 22 Aug 2022 13:21:24 -0700

Just woke up. I'll sort it out as soon as some caffeine makes it to the brain.

Mon, 22 Aug 2022 13:55:22 -0700

OK. Should be good again. Thanks for the report. That was a change my IDE recommended and I thought it was dubious, but went along with it.

These days I have to rely on the IDE to catch some things for me; because sometimes I simply can't remember why some line of code that's 5-10 years old was written a certain way.

The force_queue setting is actually correct, although I agree the code looks wrong.