Yii Framework Forum: A Bug In The Updatedialog Of Ejuidlgscolumn - Yii Framework Forum

Jump to content

Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

A Bug In The Updatedialog Of Ejuidlgscolumn Rate Topic: -----

#1 User is offline   Kagz 

  • Newbie
  • Yii
  • Group: Members
  • Posts: 1
  • Joined: 15-April 11

Posted 15 November 2012 - 02:22 PM

Hi

First I would like to say that indeed this is a great extenstion.

I think there is a bug if I try to update via EJuiDlgsColumn property of the CGridView.

array(
	//'class'=>'CButtonColumn',
	'class'=>'EJuiDlgsColumn',
	'template'=>'{update}{delete}',
	'updateDialog'=>array(
                'controllerRoute' => 'update', 
                'actionParams' => array('id'=>'$data->id'),
	        'dialogTitle' => 'Meeting Status',
		'dialogWidth' => 800, //override the value from the dialog config
		'dialogHeight' => 400,
		'closeButtonText' => 'Close',
		'closeOnAction' => true, 
		'refreshGridId' => 'meeting-status-grid',
				),  
	),


When I look at the output values through my firebug console this is what I find.

Quote

id 1?qdsClass=EJuiDlgsColumn


The qdsClass parameter and it value EJuiDlgsColumn is not sent as required, instead it is appended to the id parameter and sent.

Any workarounds?
0

#2 User is offline   AndrePierre 

  • Newbie
  • Yii
  • Group: Members
  • Posts: 1
  • Joined: 07-December 12

Posted 07 December 2012 - 12:54 PM

I have a solution for you. I downloaded the QuickDlgs extension only recently and found the same bug. I traced the code that generates this part of the parameter list to the file EJuiDlgsColumn.php, function createButtonUrl(). Find the line of code

$url .= '"?' . http_build_query($params) . '"';

Try changing that line to

$url .= '"&' . http_build_query($params) . '"';

That corrected the parameter list for me.

~ap
0

Share this topic:


Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

1 User(s) are reading this topic
0 members, 1 guests, 0 anonymous users