Eso es como minimo incorrecto, lo que estas preguntando es "Vino el campo generar_npc por por post con algún valor?" en ese if entraras por si a esa pregunta, pero nada dice eso si el request es efectivamente post, solo verifica si esta definida una variable y con el tiempo veras que hay varias formas de que ese if sea verdadero (o falso).
Código PHP:
Ver originalif($_SERVER['REQUEST_METHOD'] == "POST") {
eso lo que pregunta es si el request es POST, no pregunta por los valores enviados, si es un POST entonces se esta solicitando la accion de descargar, los parametros con lo que se solicita tal accion, son otro tema aparte.
Código PHP:
Ver original$entry=$_POST['entry'];
$modelo1=$_POST['modelo1'];
$modelo2=$_POST['modelo2'];
$modelo3=$_POST['modelo3'];
$modelo4=$_POST['modelo4'];
$nombre=$_POST['nombre'];
$subnombre=$_POST['subnombre'];
$menu_id=$_POST['menu_id'];
$minlevel=$_POST['minlevel'];
$maxlevel=$_POST['maxlevel'];
$faccion=$_POST['faccion'];
$npcflag=$_POST['npcflag'];
$escala=$_POST['escala'];
$rango=$_POST['rango'];
$damage1=$_POST['damage1'];
$damage2=$_POST['damage2'];
$unit_class=$_POST['unit_class'];
$familia=$_POST['familia'];
$tipo=$_POST['tipo'];
$resistencia1=$_POST['resistencia1'];
$resistencia2=$_POST['resistencia2'];
$resistencia3=$_POST['resistencia3'];
$resistencia4=$_POST['resistencia4'];
$resistencia5=$_POST['resistencia5'];
$resistencia6=$_POST['resistencia6'];
$mingold=$_POST['mingold'];
$maxgold=$_POST['maxgold'];
$vida=$_POST['vida'];
$mana=$_POST['mana'];
$armadura=$_POST['armadura'];
$regenvida=$_POST['regenvida'];
Eso... bueno... sin palabras. Es una tacnica mas que obsoleta que viene de PHP 4, cuando $_POST se llamaba $HTTP_POST_VARS y como es muy molesto trabajar con un nombre de variable tan largo se hacia eso.
$_POST tiene una longitud aceptable y es facil de trabajar, no tiene ningun sentido hacer eso.
ademas de que hay mejores formas, por ejemplo:
o
si lo que se busca es acortar la longitud, pero a mi criterio, ninguna tiene razon de ser salvo en casos particulares, extract por su parte es "peligrosa" si no se toman algunas precauciones.
Código PHP:
Ver originalecho "<center>"."<textarea>"."INSERT INTO .... '0');"."</textarea>"."</center>";
Eso se te indico explicitamente que lo tenias que sacar, nada tiene que ver no saber un lenguaje, si alguien te dice "los <textarea> son innecesarios, pues sólo vas a imprimir texto" y te pone un ejemplo sin los textarea, y tu los sigues dejando, claramente o no estas prestando atencion, o no te interesa, o nos tomas el pelo.
Código PHP:
Ver originalVALUES('".$entry."',
'0',
'0',
'0',
'0',
'0',
'".$modelo1."',
'".$modelo2."',
'".$modelo3."',
'".$modelo4."',
'".$nombre."',
'".$subnombre."',
NULL,
'".$menu_id."',
'".$minlevel."',
'".$maxlevel."',
'0',
'".$faccion."',
'".$npcflag."',
'0.91',
'1.14286',
'".$escala."',
'".$rango."',
'0',
'2000',
'2000',
'".$damage1."',
'".$damage2."',
'".$unit_class."',
'0',
'0',
'0',
'".$familia."',
'0',
'0',
'0',
'0',
'".$tipo."',
'0',
'0',
'0',
'0',
'".$resistencia1."',
'".$resistencia2."',
'".$resistencia3."',
'".$resistencia4."',
'".$resistencia5."',
'".$resistencia6."',
'0',
'0',
'0',
'0',
'0',
'0',
'0',
'0',
'0',
'0',
'".$mingold."',
'".$maxgold."',
'',
'0',
'3',
'1',
'".$vida."',
'".$mana."',
'".$armadura."',
'1',
'1',
'0',
'0',
'0',
'0',
'0',
'0',
'".$regenvida."',
'0',
'1',
'',
'0',
'',
Eso... bueno, siendo estrictos esta bien porque funciona, pero de las varias formas de lograr lo mismo es la mas precaria.
Yo (y cualquier otra persona) viendo esa parte del codigo sola, no tengo idea de que es lo que hace ademas de concatenar variables.
Una buena alternativa es crear una plantilla de la query:
Código PHP:
Ver original$template = "INSERT INTO `creature_template` (`entry`, `difficulty_entry_1`, `difficulty_entry_2`, `difficulty_entry_3`, `KillCredit1`, `KillCredit2`, `modelid1`, `modelid2`, `modelid3`, `modelid4`, `name`, `subname`, `IconName`, `gossip_menu_id`, `minlevel`, `maxlevel`, `exp`, `faction`, `npcflag`, `speed_walk`, `speed_run`, `scale`, `rank`, `dmgschool`, `BaseAttackTime`, `RangeAttackTime`, `BaseVariance`, `RangeVariance`, `unit_class`, `unit_flags`, `unit_flags2`, `dynamicflags`, `family`, `trainer_type`, `trainer_spell`, `trainer_class`, `trainer_race`, `type`, `type_flags`, `lootid`, `pickpocketloot`, `skinloot`, `resistance1`, `resistance2`, `resistance3`, `resistance4`, `resistance5`, `resistance6`, `spell1`, `spell2`, `spell3`, `spell4`, `spell5`, `spell6`, `spell7`, `spell8`, `PetSpellDataId`, `VehicleId`, `mingold`, `maxgold`, `AIName`, `MovementType`, `InhabitType`, `HoverHeight`, `HealthModifier`, `ManaModifier`, `ArmorModifier`, `DamageModifier`, `ExperienceModifier`, `RacialLeader`, `questItem1`, `questItem2`, `questItem3`, `questItem4`, `questItem5`, `questItem6`, `movementId`, `RegenHealth`, `mechanic_immune_mask`, `flags_extra`, `ScriptName`, `VerifiedBuild`) VALUES('[entry]', '0', '0', '0', '0', '0', '[modelo1]', '[modelo2]', '[modelo3]', '[modelo4]', '[nombre]', '[subnombre]', NULL, '[menu_id]', '[minlevel]', '[maxlevel]', '0', '[faccion]', '[npcflag]', '0.91', '1.14286', '[escala]', '[rango]', '0', '2000', '2000', '[damage1]', '[damage2]', '[unit_class]', '0', '0', '0', '[familia]', '0', '0', '0', '0', '[tipo]', '0', '0', '0', '0', '[resistencia1]', '[resistencia2]', '[resistencia3]', '[resistencia4]', '[resistencia5]', '[resistencia6]', '0', '0', '0', '0', '0', '0', '0', '0', '0', '0', '[mingold]', '[maxgold]', '', '0', '3', '1', '[vida]', '[mana]', '[armadura]', '1', '1', '0', '0', '0', '0', '0', '0', '[regenvida]', '0', '1', '', '0', '', '0');";
Donde quien la ve se da cuenta que se trata de un modelo de consulta, nada se sabe de los valores puntuales, pero esa es la idea de una plantilla. Hasta podrias poner todas las plantillas en un archivo separado y cargarlas, si dos juegos diferentes soportan "creature_template" solo cargas el template que corresponda al juego y listo, el resto del codigo no se modifica.
Eso no necesita mucha explicacion, puedes ir al manual de str_replace si intuitivamente no lo descifras o al de foreach.
Código PHP:
Ver original?>
<?php
{
for( $contador=0; $contador < count($sql); $contador++ ) echo "".$sql[$contador]."<br/>";
}
?>
Esto..
1) No tiene sentido cerrar y abrir los tag de php, de hecho una buena practica es abrirlos al inicio del archivo y no cerrarlos nunca.
hacerlo asi puede (ademas de carecer de sentido) traer problemas, ya que entre el cierre y la apertura hay un salto de linea que si trabajaras con sessiones podria darte problemas. fíjate como un detalle tan tonto te puede complicar la vida.
2) Nose que es $sql, nose que es lo que hace eso ademas de lanzar warning y notices.
Código PHP:
Ver originalheader("Content-type: application/octet-stream"); header("Content-Disposition: attachment; filename=\"codigo.sql\"\n"); echo $template;
este es codigo que vengo repitiendo ya 3 veces desde el inicio del post. en el link de @pateketrueke los primeros 20 resultados te sirven y son similares a este.