Suitecrm: Bad HTML in generated subpanels (Suite7 and SuiteR theme)

Created on 4 Apr 2017  路  6Comments  路  Source: salesagility/SuiteCRM

When sub-panels are generated, the server returns invalid HTML, which results in the browser interpreting it incorrectly.

First subpanel always truncated
screenshot - 04 04 2017 - 113908

Second and other subpanels - is ok
screenshot - 04 04 2017 - 113922

Problem in file new migrated include/SubPanel/tpls/SubPanelDynamic.tpl

Smarty template (include/SubPanel/tpls/SubPanelDynamic.tpl) was added to replace xTemplate (include/SubPanel/SubPanelDynamic.html) and pagination block moved incorrectly

include/SubPanel/SubPanelDynamic.html

<!-- BEGIN: dyn_list_view -->
<table cellpadding="0" cellspacing="0" width="100%" border="0" class="list view subpanel-table">
<tbody>
<!-- BEGIN: list_nav_row -->
{PAGINATION}

include/SubPanel/tpls/SubPanelDynamic.tpl

{$PAGINATION}
<table cellpadding="0" cellspacing="0" border="0"  data-empty=".....

PAGINATION generate in include/ListView/ListViewSubPanel.php and it is assumed that it is inserted inside the table

$html_text = '';
$html_text .= "<tr class='pagination' role='presentation'>\n";
.....
$html_text .= "</tr>\n";
$this->smartyTemplate->assign("PAGINATION", $html_text);

My Environment

  • SuiteCRM 7.8.2
  • Chrome version 56.0.2924.87 (64-bit)
Moderate Fix Proposed Bug

Most helpful comment

This problem also broke the functional of the dynamic reboot of the sub-panel. After the reboot the subpanel doubled and worked incorrectly. This had to find and correct the error. In SuiteP all ok. I've found many similar mistakes, I'll try to find the time and suggest it to the core. It is a pity that the developers no longer support the theme Suite7 and SuiteR.

All 6 comments

I had a strange issue with the Select\Create button of the first subpanel of all Detailviews of SuiteR. My issue was that the subpanel could be dragged within the view independently of the subpanel's Select button. Fixed #3356 helped to solve the issue. I hope your solution is included in the next release. Thank you!

This problem also broke the functional of the dynamic reboot of the sub-panel. After the reboot the subpanel doubled and worked incorrectly. This had to find and correct the error. In SuiteP all ok. I've found many similar mistakes, I'll try to find the time and suggest it to the core. It is a pity that the developers no longer support the theme Suite7 and SuiteR.

Yes, i have this issue Which is broking the functional of the dynamic reboot of the sub-panel after implement this fix. Please fix this issue because we are using Suite7 theme. Suite7 theme very compact than others.

When i try to apply this, it occurs - #3467.

@dursuntosun , the problem is known.
I will try to provide a solution within 2 days

Fix proposed in issue #3467

Was this page helpful?
0 / 5 - 0 ratings