BoldMinded

All add-ons currently require PHP 7.4 or greater.

On July 4th 2024 PHP 8.2 will be the new minimum requirement for all add-ons. Expect any add-on released after that date to require 8.2 or greater. Some releases may not immediately take advantage of 8.x specific features in PHP, which means you might, be able to continue using new releases in PHP 7.4, however, if you experience an error the first thing you should do is update to PHP 8.2 then create a support ticket if the error persists.

Publisher does not support the Fluid field type. Please do not contact asking when support will be available.

If you purchased an add-on from expressionengine.com, be sure to visit boldminded.com/claim to add the license to your account here on boldminded.com.

Ticket: after upgrading to Simple Grids & Tables 1.7.1, accessing Add-Ons returns an error message

Status Resolved
Add-on / Version Simple Grids & Tables 1.7.1
Severity
EE Version 7.2.17

Dan Munday

Nov 23, 2023

after upgrading to 1.7.1, accessing Add-Ons returns an error message

Fatal error: Default value for parameters with a class type can only be NULL in /Volumes/HH Comms/
HH ee7 upgrade/hh_ee7_stable/_site/system/user/addons/simple_grids/ft.simple_grid.php on line 694

The current add-on version is 1.5.5 and it works fine.
the php is 7.4.33

 

 

 

 

#1

BoldMinded (Brian)

I think on line 694, just removed mixed from the function typehint.

private function getColumnView(string $type, string $colId = ”, $data = ”): string

#2

Dan Munday

Thanks. However, editing the block which includes the 1.7.1 Simple Grid fieldtype throws an error:

Warning Use of undefined constant ‘’ - assumed ‘‘’’ (this will throw an Error in a future version of PHP) user/addons/simple_grids/ft.simple_grid.php, line 693 Severity: E_WARNING
#3

BoldMinded (Brian)

Change those fancy quotes to normal quotes.

private function getColumnView(string $type, string $colId = '', $data = ''): string
#4

Dan Munday

Hey Brian, I know why it returns the error. The code should be straight quotes instead of smart quotes. It works now.

#5

Dan Munday

Hey Brian, I am afraid that “mixed” is needed. Without this, the front end probably does not display the markup. I need to go back to 1.5.5. Do you think you can send me 1.6 for the time being? Thanks

#6

Dan Munday

I just uploaded two screenshots: 1. the correct one—running on 1.5.5 2. all the feature points are gone—running on 1.7.1

I suspect that “mixed” is necessary

#7

Dan Munday

Another test. I’ve upgraded the PHP to 8.1.13 on my local dev; keep the original 1.7.1 without revising line 694. All the feature points of the price plan are still gone on the front end.

#8

BoldMinded (Brian)

If it’s working correctly with the version you had just revert to that until I can look into this further.

#9

BoldMinded (Brian)

It would help if you shared your template code that is rendering this field.

#10

BoldMinded (Brian)

Having “mixed” as the type hint does not change the functionality, and that getColumnView method is only called in the CP, so it won’t affect the front-end rendering.

Have you tried changing anything in your template to see if it would render?

#11

Dan Munday

Hey Brian, Do you mind sending me 1.6, as I need the RTE feature? Many thanks.

#12

BoldMinded (Brian)

Not really, sorry. I’d much rather figure out what issue you’re having and resolve that instead of ignoring a potential bug.

#13

Dan Munday

I just wanted to test if 1.6 works so that we can narrow down the issue. yeah, most of the blocks using a simple grid cannot generate any content.

#14

Dan Munday

seems like condition statements are not allowed

{features}
    {if count == 1}
    <div class="features-box">{/if}
        {if section_title =='1'}
        <div class='features-section'>
            {desc}
            <ul class="pricing-features">
                {/if}
                {if section_title =='0'}
                <li class="pricing-features-item il-IconLabel il-IconLabel-align-top hs-HasSvg hs-HasSvg-18px {if without == '1'}hs-HasSvg-coral{if:else}hs-HasSvg-purple{/if}  hs-HasSvg-sp-r-small">
                    {if without == '1'}{gv_svg-cancel-outline}<span class="show-for-sr">No</span>{desc}{if:else}{gv_svg-success-outline}{desc}{/if}
                </li>
                {/if}
                {if last_item == '1'}
            </ul>
        </div>
        {/if}
        {if count == total_rows}
    </div>{/if}
    {/features}
#15

Dan Munday

I discovered that the number should not be enclosed in single quotation marks. After removing them, the content was generated successfully, but the layout appeared completely disordered. I suspect that the bug could be associated with the conditional statement.

#16

BoldMinded (Brian)

Comment has been marked private.

#17

BoldMinded (Brian)

You shouldn’t need {if section_title ==’0’} anymore, it should just work with {if section_title}

#18

Dan Munday

the feature items are shown now. However, the tab contents are missing. Not sure what happened. another bug I spot is that url_slug does not work on the field in Simple Grid

<a href="#{tab_title:url_slug}" aria-selected="true" class="tabbify-label">
#19

Dan Munday

I have some additional information to share with you.

<a href="#{tab_title:url_slug}" aria-selected="true" class="tabbify-label">

There seems to be a problem with the EE modifier where it is not rendering the correct URL slug. For example, the tab title “Full Membership” renders “#Full Membership” instead of “#full-membership”. That’s why the tabs do not work.

Once this issue is fixed, I believe all the front-end markup will be displayed correctly.

#20

BoldMinded (Brian)

Comment has been marked private.

#21

Dan Munday

It works perfectly. Many thanks for your prompt reply.

Login to reply