$yname = !empty($_POST['yname']) ? $myts->makeTboxData4Save($_POST['yname']) : "";
$ymail = !empty($_POST['ymail']) ? $myts->makeTboxData4Save($_POST['ymail']) : "";
$fname = !empty($_POST['fname']) ? $myts->makeTboxData4Save($_POST['fname']) : "";
$fmail = !empty($_POST['fmail']) ? $myts->makeTboxData4Save($_POST['fmail']) : "";
if(!isset($_POST['lid']) && isset($_GET['lid']) ) {
$lid = intval($_GET['lid']) ;
}else {
$lid = intval($_POST['lid']) ;
}
$op= '';
if (!empty($_GET['op'])) {
$op = $_GET['op'];
} elseif (!empty($_POST['op'])) {
$op = $_POST['op'];
}
zyspec wrote:
snip
Towards the bottom of the listing-p-f.php (or annonces-p-f.php depending on which version of MyAds you have) file you'll find the following code just before:
switch($op) {
You'll find:foreach ($_POST as $k => $v) {
${$k} = $v;
}
$lid = isset( $_GET['lid'] ) ? $_GET['lid'] : '' ;
if(!isset($_POST['op']) && isset($_GET['op']) ) {
$op = $_GET['op'] ;
}
Just delete these lines or comment it out with // at the beginning of each line) and replace it with:$lid = isset( $_GET['lid'] ) ? intval($_GET['lid']) : '' ;
$op = isset($_POST['op']) ? $_POST['op'] : '' ;
$op = isset($_GET['op']) ? $_GET['op'] : $op ;
$yname = isset($_POST['yname'] ? $myts->addSlashes($_POST['yname']) : '';
$ymail = isset($_POST['ymail'] ? $myts->addSlashes($_POST['ymail']) : '';
$fname = isset($_POST['fname'] ? $myts->addSlashes($_POST['fname']) : '';
$fmail = isset($_POST['fmail'] ? $myts->addSlashes($_POST['fmail']) : '';
Remember, place this BEFORE the switch($op) statement. By the way, the fix for the Job Listing module (jobs ver 1.9) is the same.
If you're only using those blocks (or is it one block?) in the News Index, maybe you might think about moving the entry for xoops_content and place it above the center blocks array.