Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

majixRepaint and database query #154

Open
AntonMezger opened this issue Dec 14, 2022 · 2 comments
Open

majixRepaint and database query #154

AntonMezger opened this issue Dec 14, 2022 · 2 comments

Comments

@AntonMezger
Copy link

AntonMezger commented Dec 14, 2022

in a renderlet with ajax majixrepaint the datasource sql gives an error in the routine fetchdata of the class_tx_mkforms_ds_db_Main. The doQuery does not give a correct result back and crashes on $iNumRows = count($rows). However on normal loading the sql statement is correctly executed.

my majixrepaint is done by the renderlet:

                <renderlet:BUTTON name="btnSearch">
                     <label>LLL:fileadmin/xml/locallang_sdg.xml:plugin.mkforms.flexform.search.label</label>
                     <onclick runat="ajax" cache="false">
                          <userobj>
                               <php><![CDATA[
                                    return array(
                                    $this->aORenderlets["pages"]->majixRepaint(),
                                    ); 
                               ]]></php>
                          </userobj>
                     </onclick>
                </renderlet:BUTTON>

I am using version 9.5.2 of mkforms.

and the error is the following

image

@hannesbochmann
Copy link
Member

Could you please check what type $rows is? Is it null? Is the query correct which get's issued 3 lines before.

@AntonMezger
Copy link
Author

sorry,

but that is not really the problem. count($rows) does not work and the solution is to calculate the count by going through the found results. Then it works for version 9. However majix does not work at all in version 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants