Problema con render_partial

rafe.cu

Como está implementado render_partial en este momento, no funciona cuando se quiere mostrar un partial de un controlador distinto del activo, debido al codigo siguiente:
function render_partial(){
$params = get_params(func_get_args());
if(isset($params)) {
$controller_name = uncamelize($params);
}elseif(preg_match("/^(.+)\/(.+)$/", $params[0], $data)) {
...
Fijense que si se pasa un parametro con nombre 'controller' para especificar que el partial es de otro controller, no se le asigna ningun valor a la variable $partial_view, y se lanza la excepcion que no se encontro la vista, esto se podría solucionar así:
...
if(isset($params)) {
$controller_name = uncamelize($params);
$partial_view = uncamelize($params);
//$partial_view = uncamelize($params);
}elseif(preg_match(
...

deivinsontejeda

Lo probaste y te funciona?
ya esto fue solucionado en la version 0.5...

Gracias...

rafe.cu

Lo probé y me funciona bien.
El fragmento de codigo que puse arriba, es de la versión 5 que la bajé del svn ayer, si te fijas en el código verás lo que digo arriba

deivinsontejeda

Perfecto realmente no hemos querido incorporarle nada nuevo a la versión 0.4x ya que seria muy recursivo cada vez que se agregue algo a la versión posterior retroceder a la versión anterior, todo lo nuevo viene para la versión 0.5...

Éxito..

rafe.cu

Al parecer no me he explicado bien, estoy probando la version 05, y no me funciona bien cuando trato de mostrar una vista parcial de otro controlador, revisé el código del render_partial, y me di cuenta que cuando se pasa un parámetro con nombre 'controller', no se le asigna un valor a la variable $partial_view, por lo que en este caso no muestra esa vista parcial, le añadí una línea de código, y funciona bien, al menos con las pruebas que hice.

deivinsontejeda

Ah ok perfecto ahora si entiendo pensé que hablabas de la versión 0.4X que bueno que se halla encontrado un Bug y que este gracias a ti se pueda solucionar voy a revisar el problema y luego realizo los cambios que tu propones, Gracias...

Éxitos...

emilio.rst

Que raro, lo acabo de probar y funciona.

Si te fijas bien:

function render_partial(){
	$params = get_params(func_get_args());

	if(isset($params['controller'])) {
		$controller_name = uncamelize($params['controller']);
	}elseif(preg_match("/^(.+)\/(.+)$/", $params[0], $data)) {
		$controller_name = $data[1];
		$partial_view = $data[2];
	} else {
		$controller_name = Router::get_controller();
	}

        /** fijate aqui  **/
	if(!isset($partial_view)) $partial_view = $params[0];

	$views_dir = Kumbia::$active_views_dir;
	$partial_file = join_path($views_dir, $controller_name, "_$partial_view.phtml");
	
	if(file_exists($partial_file)){
		if(is_array(Kumbia::$models)){
			foreach(Kumbia::$models as $model_name => $model){
				$$model_name = $model;
			}
		}
		
		if(is_subclass_of(Dispatcher::get_controller(), "ApplicationController")){
			foreach(Dispatcher::get_controller() as $var => $value) {
				$$var = $value;
			}
		}

		if(isset($params[1])) $$partial_view = $params[1];
		include $partial_file;
	} else {
		throw new kumbiaException('Kumbia no puede encontrar la vista parcial: "'.$partial_view.'"', 0);
	}
}

Fijate, la linea:
if(!isset($partial_view)) $partial_view = $params[0];

Cubre el caso del que hablas.

De cualquier forma aun estan en fase de desarrollo, hay que agregar el soporte para los archivos .tpl

Saludos.

rafe.cu

El caso que tu dices, no es del que yo hablo,
yo hablo del caso que se pase el parametro con nombre 'partial_view', y 'controller', si traceas el código, en ese caso, a $partial_view no se le asigna nada hasta llegar a la linea que tu mencionas, en ese linea, se le asigna el primer parámetro que perfectamente puede ser el nombre del controlador, y a la hora de incluir el fichero no lo encontraría

emilio.rst

Te refieres a efectuar lo siguiente

render_partial('controller: mi_controlador', 'partial_view: mi_partial');

En ese caso tienes toda la razon, no esta contemplado.

Ya lo resolvi.

Ahora se puede hacer de las siguientes maneras:

render_partial('mi_partial', $data);
render_partial('partial_view: mi_partial', $data);
render_partial('mi_controller/mi_partial', $data);
render_partial('controller: mi_controller', 'mi_partial', $data);
render_partial('controller: mi_controller', 'partial_view: mi_partial', $data);

Esta para todos los gustos.

Ya subi el cambio al svn.

Pruebenlo a ver que tal funciona.

Saludos.