post model & acp announce conflictoTema Solucionado

[Nacho1191]

Acabo de instalar el mod post model, y tengo un conflicto con el acp announce centre mod, lo cual me deja la pagina en blanco.

mi razon por la cual creo que es un conflicto con ese mod, es porque cuando tengo que editar el function posting.php veo que ya estaba modificado previamente por el announce centre.

les dejo las instrucciones del post model, y lo que tengo ya en el php editado:

Instrucciones:

Código: Seleccionar todo

Find

function generate_smilies($mode, $forum_id)

Replace with

function generate_smilies($mode, $forum_id, $acp_models = false)

find


			'body' => 'posting_smilies.html')

Replace with

			'body' => ( $acp_models ) ? 'models_smilies.html' : 'posting_smilies.html')

Find

			'U_MORE_SMILIES' 		=> append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

Replace with

			'U_MORE_SMILIES'       => ( $acp_models ) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&models=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

lo que tengo en esas lineas es:

function generate_smilies($mode, $forum_id, $acp_announce = false)

'body' => ( $acp_announce ) ? 'announcement_smilies.html' : 'posting_smilies.html')

'U_MORE_SMILIES' => ( $acp_announce ) , $ ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

como verán lo agradecería es como colocar de una manera correcta para no crear conflicto con el mod anteriormente instalado

Saludos, y gracias de antemano

P.D: Mi datos estan en la firma

[leviatan21]

lo que tengo en esas lineas es:

Yo te ayudaría, pero hay un problema :
Esta linea está mal tiene errores, no pude ser que la tengas así

Código: Seleccionar todo

'U_MORE_SMILIES' => ( $acp_announce ) , $ ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

[Nacho1191]

disculpa, muchas gracias por la rapida respuesta.
la linea correcta es:

Código: Seleccionar todo

			'U_MORE_SMILIES'       => ( $acp_announce ) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

[leviatan21]

He aquí como sería :
1)

Código: Seleccionar todo

function generate_smilies($mode, $forum_id, $acp_models = false, $acp_announce = false) 

para que quede claro, así sería como lo haría yo ( con unas lineas mas arriba y abajo ):

Código: Seleccionar todo

// Default
// function generate_smilies($mode, $forum_id)
// mod post model
// function generate_smilies($mode, $forum_id, $acp_models = false)
// acp announce centre mod
function generate_smilies($mode, $forum_id, $acp_models = false, $acp_announce = false) 

2)

Código: Seleccionar todo

			'body' => ($acp_announce) ? 'announcement_smilies.html' : (($acp_models) ? 'models_smilies.html' : 'posting_smilies.html'))

para que quede claro, así sería como lo haría yo ( con unas lineas mas arriba y abajo ):

Código: Seleccionar todo

        $template->set_filenames(array(
        // Default
        //    'body' => 'posting_smilies.html')
        // mod post model
        //    'body' => ( $acp_models ) ? 'models_smilies.html' : 'posting_smilies.html')
        // acp announce centre mod
            'body' => ($acp_announce) ? 'announcement_smilies.html' : (($acp_models) ? 'models_smilies.html' : 'posting_smilies.html'))
        );
 

3)

Código: Seleccionar todo

            'U_MORE_SMILIES'        => ($acp_announce) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : (($acp_models) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&models=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))) 

para que quede claro, así sería como lo haría yo ( con unas lineas mas arriba y abajo ):

Código: Seleccionar todo

        $template->assign_vars(array(
            'S_SHOW_SMILEY_LINK'     => true,
        // Default
        //    'U_MORE_SMILIES'         => append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))
        // mod post model
        //    'U_MORE_SMILIES'        => ($acp_announce) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))
        // acp announce centre mod
            'U_MORE_SMILIES'        => ($acp_announce) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : (($acp_models) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&models=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id)))
        );
 

[leviatan21]

Pero en algún otro lugar también habrá que hacer cambios, por lo que necesito ambos MODs para corregir la forma en que llaman a esa función

[Nacho1191]

gracias, leviatan21.
el code ya no me da error, ahora eso si acabo de ver que hay otro pequeño conflicto con el mismo announce centre, pero esta vez en una linea del posting.php

ACP CENTRE

Código: Seleccionar todo

Find

		generate_smilies('window', $forum_id);

Replace with

		if ( request_var('announce', 0) == 1 )
		{
		 generate_smilies('window', 0, 1);
		}
		else
		{
		 generate_smilies('window', $forum_id);
		}

y el pos models:

Código: Seleccionar todo

Find
		generate_smilies('window', $forum_id);

Replace with

		if ( request_var('models', 0) == 1 )
		{
		 generate_smilies('window', 0, 1);
		}
		else
		{
		 generate_smilies('window', $forum_id);
		}

sugerencias???

[Nacho1191]

disculpen por el doble post, pero creo haber integrado de buena manera ambos mods:

INTEGRACION ACP CENTRE ANNOUNCE Y POST MODELS

Abrir includes/function posting.php

Código: Seleccionar todo

Find

function generate_smilies($mode, $forum_id)

Replace with

function generate_smilies($mode, $forum_id, $acp_announce = false, $acp_models = false)

find
         'body' => 'posting_smilies.html')

Replace with

        'body' => ($acp_announce) ? 'announcement_smilies.html' : (($acp_models) ? 'models_smilies.html' : 'posting_smilies.html'))

Find

         'U_MORE_SMILIES'       => append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id))

Replace with

         'U_MORE_SMILIES'        => ($acp_announce) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&announce=1') : (($acp_models) ? append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&models=1') : append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=smilies&f=' . $forum_id)))

abrir posting.php

Código: Seleccionar todo

Find

      generate_smilies('window', $forum_id);

Replace with

      if ( request_var('announce', 0) == 1 )
      {
      if ( request_var('models', 0) == 1 )
      {
       generate_smilies('window', 0, 1);
      }
      else
      {
       generate_smilies('window', $forum_id);
      }

agradecimientos a leviatan21

[leviatan21]

ahora eso si acabo de ver que hay otro pequeño conflicto con el mismo announce centre, pero esta vez en una linea del posting.php

A eso me refería cuando dije

Pero en algún otro lugar también habrá que hacer cambios, por lo que necesito ambos MODs para corregir la forma en que llaman a esa función

Pero la forma correcta de hacer el cambio es así :

Código: Seleccionar todo

        if (request_var('announce', 0) == 1)
        {
            generate_smilies('window', 0, 0, 1);
        }
        else if (request_var('models', 0) == 1)
        {
            generate_smilies('window', 0, 1);
        }
        else
        {
            generate_smilies('window', $forum_id);
        } 

Y tal como hice antes, así lo haría yo ( con unas lineas mas arriba y abajo ) :

Código: Seleccionar todo

	case 'smilies':
		$sql = '';
	// Default
	//	generate_smilies('window', $forum_id);
		// acp announce centre mod
		if (request_var('announce', 0) == 1)
		{
			generate_smilies('window', 0, 0, 1);
		}
		// mod post model
		else if (request_var('models', 0) == 1)
		{
			generate_smilies('window', 0, 1);
		}
		else
		{
			generate_smilies('window', $forum_id);
		}
	break;