Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redirect de MF no esta funcionando #2

Closed
hunk opened this issue Mar 28, 2011 · 10 comments
Closed

Redirect de MF no esta funcionando #2

hunk opened this issue Mar 28, 2011 · 10 comments

Comments

@hunk
Copy link
Member

hunk commented Mar 28, 2011

cuando tratas de editar un post type que no esta en el sistema en la funcion edit_post_type hace un redirect, pero algo esta mal, me sale el mensaje
"Warning: Cannot modify header information - headers already sent by (output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in /Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

@gnuget
Copy link
Member

gnuget commented Mar 28, 2011

Ese error solo pasa con los post types que ya tiene wp (post and pages) con los creados por MF funciona aún bien.

Ahí lo que aplica es quitarle el edit/delete a post y a pages, en un ratón los quito.

David Valdez

On Sunday, March 27, 2011 at 6:36 PM, hunk wrote:

cuando tratas de editar un post type que no esta en el sistema en la funcion edit_post_type hace un redirect, pero algo esta mal, me sale el mensaje
"Warning: Cannot modify header information - headers already sent by (output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in /Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2

@hunk
Copy link
Member Author

hunk commented Mar 28, 2011

no pero, si aun asi, como no lo encuentra en la DB hace el redirect (cosa
que esta bien), es mas antes de eso hay una if checando si las variable
post_type existe en $_GET si no lo hace deberia mandar al dashboard pero
tampoco jala, me parece que es onda de eso de parametro init.

2011/3/27 gnuget <
reply@reply.github.com>

Ese error solo pasa con los post types que ya tiene wp (post and pages) con
los creados por MF funciona an bien.

Ah lo que aplica es quitarle el edit/delete a post y a pages, en un ratn
los quito.

David Valdez

On Sunday, March 27, 2011 at 6:36 PM, hunk wrote:

cuando tratas de editar un post type que no esta en el sistema en la
funcion edit_post_type hace un redirect, pero algo esta mal, me sale el
mensaje
"Warning: Cannot modify header information - headers already sent by
(output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in
/Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924212

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

@gnuget
Copy link
Member

gnuget commented Mar 28, 2011

Es que el mf_redirect (y el wp_safe_redirect en general) solo jalan en el hook_init, el pex es que cuando estas en edit_post_type no puedes usarlo por que cuando se ejecuta esa función el init ya se ejecuto, lo que hace wordpress en esos casos.

Creo que lo mejor es hacer una función tipo "mf_error" que escupa un error explicando cual es el pex con un diseño con una cajita roja y un link al dashboard de MF no?

David Valdez

On Sunday, March 27, 2011 at 6:53 PM, hunk wrote:

no pero, si aun asi, como no lo encuentra en la DB hace el redirect (cosa
que esta bien), es mas antes de eso hay una if checando si las variable
post_type existe en $_GET si no lo hace deberia mandar al dashboard pero
tampoco jala, me parece que es onda de eso de parametro init.

2011/3/27 gnuget

Ese error solo pasa con los post types que ya tiene wp (post and pages) con
los creados por MF funciona an bien.

Ah lo que aplica es quitarle el edit/delete a post y a pages, en un ratn
los quito.

David Valdez

On Sunday, March 27, 2011 at 6:36 PM, hunk wrote:

cuando tratas de editar un post type que no esta en el sistema en la
funcion edit_post_type hace un redirect, pero algo esta mal, me sale el
mensaje
"Warning: Cannot modify header information - headers already sent by
(output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in
/Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924212

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924231

@hunk
Copy link
Member Author

hunk commented Mar 28, 2011

mmmmmm, algo como el flash de cakephp? que salga el mensaje con link y si el
debug esta en false haga redirect con js?
asi error, este post type no esta registrado con MF bla bla bla y despues
haga reditect?

2011/3/27 gnuget <
reply@reply.github.com>

Es que el mf_redirect (y el wp_safe_redirect en general) solo jalan en el
hook_init, el pex es que cuando estas en edit_post_type no puedes usarlo por
que cuando se ejecuta esa funcin el init ya se ejecuto, lo que hace
wordpress en esos casos.

Creo que lo mejor es hacer una funcin tipo "mf_error" que escupa un error
explicando cual es el pex con un diseo con una cajita roja y un link al

dashboard de MF no?

David Valdez

On Sunday, March 27, 2011 at 6:53 PM, hunk wrote:

no pero, si aun asi, como no lo encuentra en la DB hace el redirect (cosa
que esta bien), es mas antes de eso hay una if checando si las variable
post_type existe en $_GET si no lo hace deberia mandar al dashboard pero
tampoco jala, me parece que es onda de eso de parametro init.

2011/3/27 gnuget

Ese error solo pasa con los post types que ya tiene wp (post and pages)
con
los creados por MF funciona an bien.

Ah lo que aplica es quitarle el edit/delete a post y a pages, en un
ratn
los quito.

David Valdez

On Sunday, March 27, 2011 at 6:36 PM, hunk wrote:

cuando tratas de editar un post type que no esta en el sistema en la
funcion edit_post_type hace un redirect, pero algo esta mal, me sale el
mensaje
"Warning: Cannot modify header information - headers already sent by
(output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in
/Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924212

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924231

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924289

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

@gnuget
Copy link
Member

gnuget commented Mar 28, 2011

oks, si, así como en el flash de cakephp esta chido.

PD. que chingon esta esto de poder responder directo en el mail sin entrar a github x)

David Valdez

On Sunday, March 27, 2011 at 7:25 PM, hunk wrote:

mmmmmm, algo como el flash de cakephp? que salga el mensaje con link y si el
debug esta en false haga redirect con js?
asi error, este post type no esta registrado con MF bla bla bla y despues
haga reditect?

2011/3/27 gnuget

Es que el mf_redirect (y el wp_safe_redirect en general) solo jalan en el
hook_init, el pex es que cuando estas en edit_post_type no puedes usarlo por
que cuando se ejecuta esa funcin el init ya se ejecuto, lo que hace
wordpress en esos casos.

Creo que lo mejor es hacer una funcin tipo "mf_error" que escupa un error
explicando cual es el pex con un diseo con una cajita roja y un link al

dashboard de MF no?

David Valdez

On Sunday, March 27, 2011 at 6:53 PM, hunk wrote:

no pero, si aun asi, como no lo encuentra en la DB hace el redirect (cosa
que esta bien), es mas antes de eso hay una if checando si las variable
post_type existe en $_GET si no lo hace deberia mandar al dashboard pero
tampoco jala, me parece que es onda de eso de parametro init.

2011/3/27 gnuget

Ese error solo pasa con los post types que ya tiene wp (post and pages)
con
los creados por MF funciona an bien.

Ah lo que aplica es quitarle el edit/delete a post y a pages, en un
ratn
los quito.

David Valdez

On Sunday, March 27, 2011 at 6:36 PM, hunk wrote:

cuando tratas de editar un post type que no esta en el sistema en la
funcion edit_post_type hace un redirect, pero algo esta mal, me sale el
mensaje
"Warning: Cannot modify header information - headers already sent by
(output started at /Users/sites/mf2/wp-admin/menu-header.php:89) in
/Users/sites/mf2/wp-includes/pluggable.php on line 897"

recuerdo que la probe cuando la puse, y estaba funcionado buen.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924212

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924231

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924289

Alan Edgar Garca Flores (@hunk)
ing.edgar@gmail.com | hunk@astrata.mx
hunk.com.mx | astrata.mx
Skype: i-hunk

We do what we like and we like what we do.

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_924299

@hunk
Copy link
Member Author

hunk commented Mar 28, 2011

jaja ya la implemente, igual podrias expandirla para todos tipo de mensajes

@gnuget
Copy link
Member

gnuget commented Mar 28, 2011

Muy bien, jajaja.

Si, al rato hago que puedas pasarle un parametro para que la pinte de
diferentes colores :-p

On Mon, Mar 28, 2011 at 11:36 AM, hunk <
reply@reply.github.com>wrote:

jaja ya la implemente, igual podrias expandirla para todos tipo de mensajes

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_927104

David.

@gnuget
Copy link
Member

gnuget commented Mar 29, 2011

Ya quedo, aún no cierro el issue por que creo que hay algo que vale la pena discutir.

Creo que quedaría mejor si el section,action,vars lo metemos a un arreglo tipo cake no?

Algo como:

function mf_flash( $message = "something", $url = array('section' => 'mf_dashboard', 'action' => 'main', 'vars' => array(), $type = 'error'){  } 

Así ya podemos usar algo como:

//esto
$this->mf_flash('message',array(),'error'); 

//en lugar de esto:
$this->mf_flash('message',null,null,null,'error');

@hunk
Copy link
Member Author

hunk commented Mar 29, 2011

Va, pero que url sea un array o una string ( si es string pues entendemos que ya es url completa)

hunk.com.mx
@hunk

On Mar 29, 2011, at 1:16 AM, gnugetreply@reply.github.com wrote:

Ya quedo, aún no cierro el issue por que creo que hay algo que vale la pena discutir.

Creo que quedaría mejor si el section,action,vars lo metemos a un arreglo tipo cake no?

Algo como:

function mf_flash( $message = "something", $url = array('section' => 'mf_dashboard', 'action' => 'main', 'vars' => array(), $type = 'error'){ }

Así ya podemos usar algo como:

//esto
$this->mf_flash('message',array(),'error');

//en lugar de esto:
$this->mf_flash('message',null,null,null,'error');

Reply to this email directly or view it on GitHub:
https://github.com/gnuget/Magic-Fields/issues/2#comment_929853

@gnuget
Copy link
Member

gnuget commented Apr 3, 2011

Pos ya quedo

@gnuget gnuget closed this as completed Apr 3, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants