+ <?php echo JLayoutHelper::render('joomla.html.batch.item', $dispalyData = array('extension' => 'com_banners')); ?>

typo I guess, also elsewhere in PR
$dispalyData => $displayData

More coffee then...

Also, Travis does not look happy.

Missing doc block and some spaces, minors. The code works, but before I go on and spent more time here I would like to hear if there are any objection for this. I suspect the path might not be optimal, depending on the renderer implementation (?)

I have just add some inline comments. But in generaly it looks good to me. Give me a ping if you fixed the comments above than i can test it ;)

I doubt anyone's actually doing this but if you shift something from using JHtml in full to not then it breaks overrides if someone is using JHtml's override system. Since we're being so anal retentive on B/C, it has to be said.

@mbabker if someone overrides libraries/cms/html/batch.php i guess whatever stuff the core uses want make any difference, they will write their own html part on their overrides. Am I overseeing stuff here?

Specifically the JHtml::register() API (https://github.com/mbabker/bs3-demo/blob/master/plg_system_bootstrap3/bootstrap3.php). If you don't call the JHtml method, then if someone actually registered an override, their overridden method won't get called.

Ah, yeah I did the direct call to JLayout here, which shouldn’t be really here but part of the 4

@mbabker I adjusted the code so it will be 100% B/C for the 3.x. Thanks for the input!

How to test

• login
• access the batch e.g. for articles or users
• confirm that it works
• apply this patch
• confirm that it still works

  ---

  _{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8088.}

I have tested this item successfully on 009786a

Test erfolgreich durchgeführt

I have tested this item successfully on 009786a

it works for my

Thank you @dgt41 and testers! Merged.