Fixed rowid placeholder

[diego_godina]

Hey guys,

Is the {rowid} placeholder still valid for the J4? As soon as I updated this placeholder it stopped working.

Furthermore, are there new rules for using placeholders in modules?

 

[troester]

As soon as I updated this placeholder

How? Where?

 

[diego_godina]

When I updated Joomla, PlaceHolder stopped working on lists and list modules.

 

[troester]

Where do you use it? Steps to replicate...

 

[diego_godina]

I have a list and I want to put a module inside the details. So I created a fabrik list module and put it in a display element to show only in details. About the module configuration, I added some pre filters using the placeholder {rowid} and the format table___name_element

 

[troester]

Still not sure what you put where.

But
Do you have SEF enabled?
Is it working without SEF?

 

[diego_godina]

Yes! Without SEF the {rowid} works!! Do you know why?

 

[troester]

Some routing issue, it seems with SEF on rowid is missing in $_POST or so.

Was it working with SEF on in J!3?

 

[diego_godina]

Yes, it was working. How could I solve this issue?

 

[troester]

I don't know if there's a routing issue or if J!4 has changed here.
Can you try to add in \libraries\fabrik\fabrik\fabrik\Helpers\Worker.php after line 804

                //J!4 & SEF: $_REQUEST is empty, take also inputVars
                $app = Factory::getApplication();
                $inputVars = $app->getInput()->getArray();
                $inputVars = $f->clean($inputVars,'string');

 

[troester]

Oops, most important line is missing

                //J!4 & SEF: $_REQUEST is empty, take also inputVars
                $app = Factory::getApplication();
                $inputVars = $app->getInput()->getArray();
                $inputVars = $f->clean($inputVars,'string');
$searchData = is_null($searchData) ? $inputVars : array_merge($inputVars, $searchData);

 

[jcc]

I was getting ready to report this same issue.

I have a form with footer text as

{fabrik view=list id=2 showfilters=0 show-title=0 t2___t1id=[rowid]}

With SEF enabled, the resulting query did not substitute the rowid placeholder. Query restriction reported as:

WHERE ( `t2`.`t1id` = '{rowid}' ) LIMIT 10

Your suggested change corrected the issue and restriction now substituting:

WHERE ( `t2`.`t1id` = 1 ) LIMIT 10