Error after upgrading from Github

Status
Not open for further replies.
cchiacchierini

cchiacchierini

Member
ciderweekny.com
fabrik 3.0.8
Joomla 2.5.11

I updated from github today using fabrik-master and now I have 2 problems.

PROBLEM 1. I get the following error at the bottom of my site:

Code: 
Warning: Missing argument 2 for PlgFabrik_Form::usesSession_result(), called in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/pluginmanager.php on line 585 and defined in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/plugin-form.php on line 439
 
Warning: Missing argument 2 for PlgFabrik_Form::usesSession_result(), called in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/pluginmanager.php on line 585 and defined in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/plugin-form.php on line 439
 
Warning: Missing argument 2 for PlgFabrik_Form::usesSession_result(), called in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/pluginmanager.php on line 585 and defined in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/plugin-form.php on line 439
 
Warning: Missing argument 2 for PlgFabrik_Form::usesSession_result(), called in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/pluginmanager.php on line 585 and defined in /home/xxx/public_html/ciderweekny.com/components/com_fabrik/models/plugin-form.php on line 439

PROBLEM 2: http://www.ciderweekny.com/cider-makers returns image links instead of images.

Any ideas would be greatly appreciated.
 
1 - should be fixed as of this morning

For the image - what options do you have set for the element, e.g. the 'show media in list' option - is that set to 'yes' ?
 
The Git update from today fixed the error. Thanks for that.

Still need help with the images not showing when you have a moment.
 
I can't replicate this, images show OK when set to display in the list.

Are you using the 'download script' option?

Maybe post a screenshot of your upload element settings, so i can exactly replicate your config.

Also, can you quickly try with an out-of-box template, make sure this isn't some artifact of your custom template.

-- hugh
 
cheesegrits said:
I can't replicate this, images show OK when set to display in the list.

Are you using the 'download script' option?

Maybe post a screenshot of your upload element settings, so i can exactly replicate your config.

Also, can you quickly try with an out-of-box template, make sure this isn't some artifact of your custom template.

-- hugh
Click to expand...


I tried a out of box template and still had the same problem with the images no longer showing.

I uploaded the settings image for you to see.
 

Attachments

  • fabriklogoelementsettings.png
    fabriklogoelementsettings.png
    93.1 KB · Views: 279
I found problem #3 since updating from Github: when I added a checkbox field (yearsparticipated) to the locations table, I got the error below at the top of the locationsadmin list at http://ciderweekny.com/manage-locations. Note that this page is behind a password and can be accessed by going to ciderweekny.com/admin-page and logging in with the credentials I provided on the my sites page. The error also shows when viewing the locations admin list in the back end.

Code: 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: strrpos() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 111
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 112
 
Warning: substr() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/joomla/mail/helper.php on line 113
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
Warning: mb_strtolower() expects parameter 1 to be string, array given in /home/xxx/public_html/ciderweekny.com/libraries/phputf8/mbstring/core.php on line 121
 
the mail error is pretty odd, I don't see why or where we would be triggered by a checkbox element.
I think your site has caching on, so its hard to test this, as my admin account doesnt have access to edit the global configuration, could you bump me up to super admin please?
 
I can see these error, too.
I could break it down to a field element with isemail validation.
It's ok if element is disabled, but after deleting the validation (+element enabled + browser cache cleared; Joomla cache is off) the errors are back:confused:

Errors are on list view!

Edit1:
it's not the ismail validation; in an other list the errors are gone after disabling an field element with notempty validation.

Edit2:
found the reason:
errors if element's guess link type = yes
 
  • Like
Reactions: rob
rob said:
the mail error is pretty odd, I don't see why or where we would be triggered by a checkbox element.
I think your site has caching on, so its hard to test this, as my admin account doesnt have access to edit the global configuration, could you bump me up to super admin please?
Click to expand...

I went ahead and added you as a super admin on the site. Thanks for following up on this.

One more thing: I added a text area field also called 'notes' to the 'LocationsAdmin' list as well; that resulted in more errors on top of the previous ones.
 
troester said:
I can see these error, too.
I could break it down to a field element with isemail validation.
It's ok if element is disabled, but after deleting the validation (+element enabled + browser cache cleared; Joomla cache is off) the errors are back:confused:

Errors are on list view!

Edit1:
it's not the ismail validation; in an other list the errors are gone after disabling an field element with notempty validation.

Edit2:
found the reason:
errors if element's guess link type = yes
Click to expand...

Not sure where to find the guess link type.
 
its the 'Establishment' element that was causing the issue, as Troester says the element's 'guess link type' option was to blame, I've made a tiny tweak to the code to fix that on your site and in github
the errors should now be gone.
 
It's the "Media in form" setting mixing up the list setting (I assume this is a bug).
Show media in list=yes -> settings from "Media in form" are taken (which is NO in your case)

I think it should be: list = yes -> show first thumbnail, then cropped, then full (independend of form settings)
 
All three original problems are fixed, but a new problem has arisen though with the latest github. When activating the mediabox pop up on http://ciderweekny.com/locations by clicking on any of the locations in the list, a new error occurs:

Code: 
Notice: Trying to get property of non-object in /home/xxxx/public_html/ciderweekny.com/components/com_fabrik/views/form/tmpl/contacts_customlocationsdetailscustom/default_group.php on line 104
 
Notice: Trying to get property of non-object in /home/xxxx/public_html/ciderweekny.com/components/com_fabrik/views/form/tmpl/contacts_customlocationsdetailscustom/default_group.php on line 108
 
Notice: Trying to get property of non-object in /home/xxxx/public_html/ciderweekny.com/components/com_fabrik/views/form/tmpl/contacts_customlocationsdetailscustom/default_element.php on line 22
 
Notice: Trying to get property of non-object in /home/xxxx/public_html/ciderweekny.com/components/com_fabrik/views/form/tmpl/contacts_customlocationsdetailscustom/default_element.php on line 24

Here is the default_element.php file:

Code: 
<?php
/*
* This is where you will do your main template modifications.
*
*/
?>
 
<?php
/*
* This code sets up your first group.
*/
    reset($this->groups);
    $this->group = current($this->groups);
    $this->elements = $this->group->elements;
?>
 
<?php
/*
* Now we output the first group.  First a standard frameset, with id and
* style info.
*/
?>
 
    <fieldset class="fabrikGroup" id="group<?php echo $this->group->id;?>" style="<?php echo $this->group->css;?>">
    <legend><?php /*echo $this->group->title;*/?></legend>
 
    <?php if ($this->group->intro !== '') {?>
    <div class="groupintro"><?php echo $this->group->intro ?></div>
    <?php }?>
<?php
/*
* This is the meat of the customization, that allows you to place and
* format your elements on the page.  In this example, we're pretty much
* just duplicating the standard 'default' template layout, but doing it
* by placing each individual element, one by one.  You can get as creative
* as you want in your HTML formatting.
*
* The important thing is the two PHP lines for each element:
*
*    $this->element = $this->elements['short_element_name'];
*    echo $this->loadTemplate('element');
*
* ... which is what actually renders each individual element.  Note
* that this is one of the few places in Fabrik where you use the short
* element name (like 'first_name') instead of the full element name
* (like 'jos__fb_contact_sample___first_name').
*/
?>       
        <table>
       
        <tr>
        <td VALIGN=TOP width="70%">
            <?php
            $this->element = $this->elements['establishmentname'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['establishmentaddressstreet'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['estabconcatenatecitystatezip'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['establishmentphone'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['establishmentwebsitelink'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['establishmenthours'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['establishmentdescription'];
            echo $this->loadTemplate('element');
            ?>
        </td>
        <td VALIGN=TOP>
            <?php
            $this->element = $this->elements['establishmentmap'];
            echo $this->loadTemplate('element');
            ?>
            <?php
            $this->element = $this->elements['googlemaplink'];
            echo $this->loadTemplate('element');
            ?>
        </td>
        </tr>
        </table>           
       
    </fieldset>
 
<?php
/*
* This chunk of code selects the next (in this case second) group ... for
* each group you want to work with, you need to put this chunk of code
* to set up $this->group for the display code.
*/
    $this->group = next($this->groups);
    $this->elements = $this->group->elements;
?>
 
    <fieldset class="fabrikGroup" id="group<?php echo $this->group->id;?>" style="<?php echo $this->group->css;?>">
        <legend><?php echo $this->group->title;?></legend>
 
                <div class="example">
                        <?php
                        $this->element = $this->elements['message'];
                        echo $this->loadTemplate('element');
                        ?>
                </div>
 
    </fieldset>
 
 
<?php
/* This must be the last thing that happens in this template.  It adds
* all hidden elements to the form, and also finds any non-hidden elements
* which haven't been displayed, and adds them as hidden elements (this
* prevents JavaScript errors where element handler code can't find the actual
* DOM structures for their elements)
*/
    /*echo $this->loadTemplate('group_hidden');*/
?>

Here is the default_element.php file:

Code: 
<?php
/*
This part of the template is what actually renders each individual element.  You will be loading this
template multiple times (once for each element you want to display) from your default_group.php file.
 
You probably won't need to edit this file - most changes you want can probably be done
by overriding the template_css.php file in your J template html overrides folder
 
If you do edit this file, make sure you use the same parts of the element this example uses,
i.e. the same class definitions, etc.
*/
?>
    <?php if ($this->tipLocation == 'above') {
        echo '<div>' . $element->tipAbove . '</div>';
    }?>
    <?php
    // as this is a custom template, you may want to exclude $this->element->colum
    // which is the inline css required to abide by the group's column set up
    ?>
    <div <?php echo @$this->element->column;?> class="<?php echo $this->element->containerClass;?>">
        <?php //*echo $this->element->label;*//?>
        <?php echo $this->element->errorTag; ?>
        <div class="fabrikElement">
            <?php echo $this->element->element;?>
        </div>
        <?php if ($this->tipLocation == 'side') {
        echo $element->tipSide;
    }?>
        <div style="clear:both"></div>
    </div>
    <?php if ($this->tipLocation == 'below') {
        echo '<div>' . $element->tipBelow . '</div>';
    }?>
 
    <?php
    $this->element->rendered = true;
    ?>

Thanks for sticking with me on this one.
 
Status
Not open for further replies.

Useful links

  • Create a Fabrik Account (Downloads)
  • Create a Forum Account
  • Fabrik Wiki
  • Fabrik 4 Changelog /Install /Upgrade
    • We are in need of some funding.
    More details.

    Thank you.

    Members online

    Total: 172 (members: 2, guests: 170)
    Back
    Top