From 297f57fe3e9cd79ac45bed6f5b6f68dcc0b30b0f Mon Sep 17 00:00:00 2001 From: Erik Amaru Ortiz Date: Fri, 23 Mar 2012 19:21:38 -0400 Subject: [PATCH] BUG 8080 "when we do right click on an step there is a javasc..." SOLVED - The main problem was on the onclick user functions were being setting up on paged tables headers, (with gulliver vars like @QSTEP_UID, etc) that were not replacing properly beuase are headers and not normal rows!!, that was wrong and causing the famous problem of "Conditional Compilation Error" on IE - Finally this problem is solved leaving the onclick events for those hheaders only with doSort() at propeltable function. --- workflow/engine/classes/class.propelTable.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/workflow/engine/classes/class.propelTable.php b/workflow/engine/classes/class.propelTable.php index ac22a6a4e..eaa21672f 100755 --- a/workflow/engine/classes/class.propelTable.php +++ b/workflow/engine/classes/class.propelTable.php @@ -318,8 +318,12 @@ class propelTable $this->tpl->assign( "href" , $this->style[$r]['href']); if (isset($this->style[$r]['onsort'])) $this->tpl->assign( "onsort" , htmlentities( $this->style[$r]['onsort'] , ENT_QUOTES, 'UTF-8' ) ); + + /* BUG 8080 - erik: don't setup onclick on page table header, doesn't have sense and causing problems if (isset($this->style[$r]['onclick'])) $this->tpl->assign( "onclick" , htmlentities( $this->style[$r]['onclick'] , ENT_QUOTES, 'UTF-8' ) ); + */ + if (isset($this->style[$r]['colWidth'])) $this->tpl->assign( "width" , $this->style[$r]['colWidth'] ); if (isset($this->style[$r]['colWidth']))