<![CDATA[ Failed Formit 4.1.0 upgrade and now no manager! - My Forums]]> https://forums.modx.com/thread/?thread=104259 <![CDATA[Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560790
Any ideas what is going on and how to fix???

Looks like same issues here > https://github.com/Sterc/FormIt/issues/183]]> dubbs Aug 15, 2018, 04:44 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560790 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-562150
For anyone else landing here, be careful what brackets you change. For me, I changed everything in dubbs' list and it didn't work. I had to then plough through the error log changing one line at at time. (I guess if, unlike me, you know your square brackets from your round you can scoot through the list).

Anyway, upshot is that my site's back to normal, with formit 3 back installed. Scary though - I've never had a problem updating Modx Extras, ever. I'll be a little more cautious from now on.]]> faolie Oct 17, 2018, 03:00 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-562150 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-561401 Quote from: dubbs at Aug 16, 2018, 08:33 PM
Quote from: BobRay at Aug 16, 2018, 07:59 PM
Either the FormIt code, or the server requirements here should be changed, imo.

I would vote for the former. There's no reason to use the new-style array syntax in MODX. The old-style syntax is not going away and, as we've seen, using the new style will actually crash some MODX user's sites. AFAIK, the new style is strictly cosmetic and the old style is not deprecated.


Thanks for the reply Bob - if its not deprecated - then yeh, I tend to agree - it's not a worthy change... breaking the manager with no clue as to why is a real pain to debug!

Just had the same issue dubbs, thanks for the fix, I was able to increase the php version to 7.0.31]]> paulp Sep 05, 2018, 02:59 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-561401 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560839 Quote from: BobRay at Aug 16, 2018, 07:59 PM
Either the FormIt code, or the server requirements here should be changed, imo.

I would vote for the former. There's no reason to use the new-style array syntax in MODX. The old-style syntax is not going away and, as we've seen, using the new style will actually crash some MODX user's sites. AFAIK, the new style is strictly cosmetic and the old style is not deprecated.


Thanks for the reply Bob - if its not deprecated - then yeh, I tend to agree - it's not a worthy change... breaking the manager with no clue as to why is a real pain to debug!]]> dubbs Aug 16, 2018, 08:33 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560839 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560834 server requirements here should be changed, imo.

I would vote for the former. There's no reason to use the new-style array syntax in MODX. The old-style syntax is not going away and, as we've seen, using the new style will actually crash some MODX user's sites. AFAIK, the new style is strictly cosmetic and the old style is not deprecated.

]]> BobRay Aug 16, 2018, 07:59 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560834 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager! (Best Answer)]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560802 Quite a few files to update.. but seems to be working now...

I had to amend code in these locations:

core/components/formit/controllers/home.class.php
core/components/formit/model/formit/formitform.class.php
core/components/formit/src/Formit.php
core/components/formit/src/Formit/Dictionary.php
core/components/formit/src/Formit/Hook.php
core/components/formit/src/Formit/Request.php
core/components/formit/src/Formit/Hook/Email.php
core/components/formit/src/Formit/Hook/Redirect.php
core/components/formit/src/Formit/Hook/Saveform.php]]> dubbs Aug 15, 2018, 08:36 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560802 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560796 mod_fcgid: stderr: PHP Parse error: syntax error, unexpected '[' in /core/components/formit/src/FormIt.php on line 24

Line 24 is:
public $config = [];

How can i edit this to make it PHP 5.3.3 compliant?]]> dubbs Aug 15, 2018, 08:12 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560796 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560794
Looking in server log - I see the same issue reported here > https://forums.modx.com/thread/103766/formit-4-0-update-broke-manager-login]]> dubbs Aug 15, 2018, 07:59 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560794 <![CDATA[Re: Failed Formit 4.1.0 upgrade and now no manager!]]> https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560793 removed formit from the core>packages directory... still no joy...????]]> dubbs Aug 15, 2018, 07:56 PM https://forums.modx.com/thread/104259/failed-formit-4-1-0-upgrade-and-now-no-manager#dis-post-560793