|
From: <luc...@us...> - 2013-07-17 18:17:34
|
Revision: 11835
http://sourceforge.net/p/xoops/svn/11835
Author: luciorota
Date: 2013-07-17 18:17:28 +0000 (Wed, 17 Jul 2013)
Log Message:
-----------
FIXED ERRORS
Modified Paths:
--------------
XoopsModules/wfdownloads/trunk/wfdownloads/admin/indexpage.php
XoopsModules/wfdownloads/trunk/wfdownloads/admin/mirrors.php
XoopsModules/wfdownloads/trunk/wfdownloads/admin/reviews.php
Modified: XoopsModules/wfdownloads/trunk/wfdownloads/admin/indexpage.php
===================================================================
--- XoopsModules/wfdownloads/trunk/wfdownloads/admin/indexpage.php 2013-07-17 11:48:20 UTC (rev 11834)
+++ XoopsModules/wfdownloads/trunk/wfdownloads/admin/indexpage.php 2013-07-17 18:17:28 UTC (rev 11835)
@@ -59,7 +59,6 @@
wfdownloads_xoops_cp_header();
$indexAdmin = new ModuleAdmin();
echo $indexAdmin->addNavigation('indexpage.php');
- //wfdownloads_adminMenu(1, _AM_WFD_INDEXPAGE);
echo "<fieldset><legend>" . _AM_WFD_IPAGE_INFORMATION . "</legend>\n";
echo "<div>" . _AM_WFD_MINDEX_PAGEINFOTXT . "</div>\n";
Modified: XoopsModules/wfdownloads/trunk/wfdownloads/admin/mirrors.php
===================================================================
--- XoopsModules/wfdownloads/trunk/wfdownloads/admin/mirrors.php 2013-07-17 11:48:20 UTC (rev 11834)
+++ XoopsModules/wfdownloads/trunk/wfdownloads/admin/mirrors.php 2013-07-17 18:17:28 UTC (rev 11835)
@@ -71,27 +71,31 @@
case "mirror.edit" :
$mirror_id = wfdownloads_CleanVars($_REQUEST, 'mirror_id', 0, 'int');
- $ok = wfdownloads_CleanVars($_POST, 'ok', false, 'boolean');
if (!$mirror = $mirror_handler->get($mirror_id)) {
redirect_header($currentFile, 4, _AM_WFD_ERROR_MIRRORNOTFOUND);
exit();
}
- if ($ok == true) {
- $mirror->setVar('title', trim($_POST['title']));
- $mirror->setVar('homeurl', formatURL(trim($_POST['homeurl'])));
- $mirror->setVar('location', trim($_POST['location']));
- $mirror->setVar('continent', trim($_POST['continent']));
- $mirror->setVar('downurl', formatURL(trim($_POST['downurl'])));
- $mirror->setVar('submit', intval($_POST['approve']));
- $mirror_handler->insert($mirror);
- redirect_header($currentFile, 1, _AM_WFD_MIRROR_MIRROR_UPDATED);
+ wfdownloads_xoops_cp_header();
+ $sform = $mirror->getForm();
+ $sform -> display();
+ xoops_cp_footer();
+ break;
+
+ case "mirror.save" :
+ $mirror_id = wfdownloads_CleanVars($_REQUEST, 'mirror_id', 0, 'int');
+ if (!$mirror = $mirror_handler->get($mirror_id)) {
+ redirect_header($currentFile, 4, _AM_WFD_ERROR_MIRRORNOTFOUND);
exit();
- } else {
- wfdownloads_xoops_cp_header();
- $sform = $mirror->getForm();
- $sform -> display();
- xoops_cp_footer();
}
+ $mirror->setVar('title', trim($_POST['title']));
+ $mirror->setVar('homeurl', formatURL(trim($_POST['homeurl'])));
+ $mirror->setVar('location', trim($_POST['location']));
+ $mirror->setVar('continent', trim($_POST['continent']));
+ $mirror->setVar('downurl', formatURL(trim($_POST['downurl'])));
+ $mirror->setVar('submit', (int)($_POST['approve']));
+ $mirror_handler->insert($mirror);
+ redirect_header($currentFile, 1, _AM_WFD_MIRROR_MIRROR_UPDATED);
+ exit();
break;
case "mirrors.list" :
@@ -103,7 +107,7 @@
$start_waiting = wfdownloads_CleanVars($_GET, 'start_waiting', 0, 'int');
$start_published = wfdownloads_CleanVars($_GET, 'start_published', 0, 'int');
- $criteria_waiting = new Criteria("submit", false);
+ $criteria_waiting = new Criteria("submit", 0); // false
$waiting_count = $mirror_handler->getCount($criteria_waiting);
$criteria_waiting->setSort("date");
$criteria_waiting->setOrder("DESC");
@@ -111,7 +115,7 @@
$criteria_waiting->setStart($start_waiting);
$mirrors_waiting = $mirror_handler->getObjects($criteria_waiting);
- $criteria_published = new Criteria("submit", true);
+ $criteria_published = new Criteria("submit", 1); // true
$published_count = $mirror_handler->getCount($criteria_published);
$criteria_published->setSort("date");
$criteria_published->setOrder("DESC");
Modified: XoopsModules/wfdownloads/trunk/wfdownloads/admin/reviews.php
===================================================================
--- XoopsModules/wfdownloads/trunk/wfdownloads/admin/reviews.php 2013-07-17 11:48:20 UTC (rev 11834)
+++ XoopsModules/wfdownloads/trunk/wfdownloads/admin/reviews.php 2013-07-17 18:17:28 UTC (rev 11835)
@@ -58,7 +58,7 @@
exit();
}
if ($ok == true) {
- $review->setVar('submit', 1);
+ $review->setVar('submit', 1); // true
$review_handler->insert($review);
redirect_header($currentFile, 1, sprintf( _AM_WFD_REV_REVIEW_UPDATED, $review->getVar('title')));
exit();
@@ -71,27 +71,31 @@
case "review.edit":
$review_id = wfdownloads_CleanVars($_REQUEST, 'review_id', 0, 'int');
- $ok = wfdownloads_CleanVars($_POST, 'ok', false, 'boolean');
if (!$review = $review_handler->get($review_id)) {
redirect_header($currentFile, 4, _AM_WFD_ERROR_REVIEWNOTFOUND);
exit();
}
- if ($ok == true) {
- $review->setVar('title', trim($_POST['title']));
- $review->setVar('review', trim($_POST['review']));
- $review->setVar('rated', (int)$_POST['rated']);
- $review->setVar('submit', (int)$_POST['approve']);
- $review_handler->insert($review);
- redirect_header($currentFile, 1, _AM_WFD_REV_REVIEW_UPDATED);
+ wfdownloads_xoops_cp_header();
+ $indexAdmin = new ModuleAdmin();
+ echo $indexAdmin->addNavigation($currentFile);
+ $sform = $review->getForm();
+ $sform -> display();
+ xoops_cp_footer();
+ break;
+
+ case "review.save":
+ $review_id = wfdownloads_CleanVars($_REQUEST, 'review_id', 0, 'int');
+ if (!$review = $review_handler->get($review_id)) {
+ redirect_header($currentFile, 4, _AM_WFD_ERROR_REVIEWNOTFOUND);
exit();
- } else {
- wfdownloads_xoops_cp_header();
- $indexAdmin = new ModuleAdmin();
- echo $indexAdmin->addNavigation($currentFile);
- $sform = $review->getForm();
- $sform -> display();
- xoops_cp_footer();
}
+ $review->setVar('title', trim($_POST['title']));
+ $review->setVar('review', trim($_POST['review']));
+ $review->setVar('rated', (int)$_POST['rated']);
+ $review->setVar('submit', (int)$_POST['approve']);
+ $review_handler->insert($review);
+ redirect_header($currentFile, 1, _AM_WFD_REV_REVIEW_UPDATED);
+ exit();
break;
case "reviews.list":
@@ -99,7 +103,7 @@
$start_waiting = wfdownloads_CleanVars($_GET, 'start_waiting', 0, 'int');
$start_published = wfdownloads_CleanVars($_GET, 'start_published', 0, 'int');
- $criteria_waiting = new Criteria("submit", false);
+ $criteria_waiting = new Criteria("submit", 0); // false
$waiting_count = $review_handler->getCount();
$criteria_waiting->setSort("date");
$criteria_waiting->setOrder("DESC");
@@ -107,7 +111,7 @@
$criteria_waiting->setStart($start_waiting);
$reviews_waiting = $review_handler->getObjects($criteria_waiting);
- $criteria_published = new Criteria("submit", true);
+ $criteria_published = new Criteria("submit", 1); // true
$published_count = $review_handler->getCount();
$criteria_published->setSort("date");
$criteria_published->setOrder("DESC");
|