1. Fabrik 3.9 has been released. If you have updated Joomla to 3.9, this is a required update.
    Dismiss Notice

Error After Joomla Update to 3.9.16

Discussion in 'Community' started by startpoint, Mar 10, 2020.

  1. startpoint

    startpoint Active Member

    Level: Community
    In edit list->details->layout
    Fields: Front-end template, Admin template, PDF template are empty.
    In Edit form->Layout
    Fields: every dropdown for select template is empty
    Warning: Invalid argument supplied for foreach() in /administrator/components/com_fabrik/models/fields/fabriktemplate.php on line 60

    PHP - 7.2
    Fabrik - Latest Github
     
    Last edited: Apr 10, 2020
    startpoint, Mar 10, 2020
    #1
  2. troester

    troester Well-Known Member Staff Member

    Level: Community
    I've pointed it to @cheesegrits
    Similar error
    /administrator/components/com_fabrik/models/fields/fabrikfolderlist.php on line 54
    e.g. if editing a viz
     
    troester, Mar 10, 2020
    #2
  3. startpoint

    startpoint Active Member

    Level: Community
    Change Forum Topic to prevent update from others until problem resolve, if you can.
     
    startpoint, Mar 10, 2020
    #3
  4. troester

    troester Well-Known Member Staff Member

    Level: Community
    Looks like a bug in Joomla.

    They changed libraries\joomla\form\fields\folderlist.php to

    $path = ltrim($this->directory, '/');
    if (!is_dir($path))....

    but after ltrim is_dir() is false and getOptions is returning NULL
     
    troester, Mar 10, 2020
    #4
  5. ereyes

    ereyes New Member

    Level: Community
    So the correction would be in the hands of Joomla? Right ?
     
    ereyes, Mar 12, 2020
    #5
  6. troester

    troester Well-Known Member Staff Member

    Level: Community
    It's fixed in GitHub (Fabrik is skipping the Joomla issue).
     
    troester, Mar 19, 2020
    #6
  7. ereyes

    ereyes New Member

    Level: Community
    Excellent !
    Thank you.
     
    ereyes, Mar 23, 2020
    #7
  8. flinx

    flinx New Member

    Level: Community
    Hi, since for various reasons I cannot update via github for now I have restored the code of the joomla library bringing it back to the one previous to 3.9.16 do you think I will have problems from other parts of the site? do you think joomla will modify this bug?
     
    flinx, Mar 26, 2020
    #8
  9. troester

    troester Well-Known Member Staff Member

    Level: Community
    troester, Mar 26, 2020
    #9
  10. flinx

    flinx New Member

    Level: Community
    Thanks, I did as recommended by troester and it seems to be working well, thanks again for avoiding me to act in a way that is surely wrong and dangerous for the site.
     
    flinx, Mar 26, 2020
    #10
  11. hleithner

    hleithner New Member

    Level: Community
    Hi,

    can someone please test Joomla Pull request 28393 found on github so I can merge it in Joomla! 3.9.17 to fix the issue for all extensions?

    thx

    Harald Leithner
     
    hleithner, Apr 10, 2020
    #11
    troester likes this.
  12. troester

    troester Well-Known Member Staff Member

    Level: Community
    troester, Apr 10, 2020
    #12
  13. hleithner

    hleithner New Member

    Level: Community
    Could you please write this as comment in github.

    thx
     
    hleithner, Apr 10, 2020
    #13
  14. troester

    troester Well-Known Member Staff Member

    Level: Community
    Done.
     
    troester, Apr 10, 2020
    #14
  15. hleithner

    hleithner New Member

    Level: Community
    Perfect thx
     
    hleithner, Apr 10, 2020
    #15
  16. mattsh

    mattsh Member

    Level: Community
    @troester Is "your" patch also needed for Joomla 3.9.18?
    Running 30+ joomla sites....

    Thanks
    Matt
     
    mattsh, Apr 27, 2020
    #16
  17. troester

    troester Well-Known Member Staff Member

    Level: Community
    No, they corrected the bug in J!3.9.17

    But if you added the patch for J!3.9.16 it won't hurt with J!3.9.18.
     
    troester, Apr 27, 2020
    #17
    mattsh likes this.

Share This Page