completed dashboard news administration pages

git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@1795 c91229c3-7414-0410-bfa2-8a42b809f60b

completed dashboard news administration pages
git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@1795 c91229c3-7414-0410-bfa2-8a42b809f60b
michael
1 parent 345bdee5
Showing 6 changed files with 289 additions and 71 deletions
presentation/lookAndFeel/knowledgeTree/administration/news/addNewsBL.php
presentation/lookAndFeel/knowledgeTree/administration/news/editNewsBL.php
presentation/lookAndFeel/knowledgeTree/administration/news/manageNewsBL.php
presentation/lookAndFeel/knowledgeTree/administration/news/newsUI.inc
presentation/lookAndFeel/knowledgeTree/administration/news/previewNewsBL.php
presentation/lookAndFeel/knowledgeTree/administration/news/removeNewsBL.php
+<?php
+
+require_once("../../../../../config/dmsDefaults.php");
+require_once("$default->fileSystemRoot/lib/dashboard/DashboardNews.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternMainPage.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternCustom.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternTableSqlQuery.inc");
+require_once("$default->uiDirectory/administration/news/newsUI.inc");
+
+/**
+ * $Id$
+ *  
+ * Business logic for adding a news item
+ *
+ * Licensed under the GNU GPL. For full terms see the file DOCS/COPYING.
+ *
+ * @version $Revision$
+ * @author Michael Joseph <michael@jamwarehouse.com>, Jam Warehouse (Pty) Ltd, South Africa
+ * @package presentation.lookAndFeel.knowledgeTree.administration.news
+ */
+
+if (checkSession()) {	
+	global $default;
+			
+    // instantiate my content pattern
+    $oContent = new PatternCustom();
+    
+    // check that we have all the required parameters
+    if ($fStore) {
+    	// construct the news object
+    	$oDashboardNews = new DashboardNews($fSynopsis, $fBody, $fRank);
+
+	    if (isset($fSynopsis) && isset($fBody) && isset($fRank)) {
+			// if we have a new image
+			if (strlen($_FILES['fImage']['name']) > 0) {
+				// return the size of the image
+				$aSize = getimagesize($_FILES['fImage']['tmp_name']);
+				// don't accept it if it isn't an image
+				if (!$aSize) {
+					$default->log->error("addNewsBL.php attempted to upload a non-image:" . $_FILES['fImage']['tmp_name']);
+					$oContent->setHtml(renderAddNewsPage($oDashboardNews, "You may only upload an image file."));
+				} else {
+					// we have an image, now check the size
+					$iImgWidth = $aSize[0];
+					$iImgHeight = $aSize[1];
+					if ($oDashboardNews->checkImageSize($iImgWidth, $iImgHeight)) {
+						// size is fine, so set it
+						$default->log->info("setting image file=" . $_FILES['fImage']['tmp_name']);
+						$oDashboardNews->setImageFile($_FILES['fImage']['tmp_name']);
+						
+						// store it
+			    		if ($oDashboardNews->create()) {
+			    			$default->log->info("addNewsBL.php successfully created dashboard news id=" . $oDashboardNews->getID());
+			    			// redirect to view page
+			    			redirect("$default->rootUrl/control.php?action=viewNews");
+			    		} else {
+			    			// insert failed
+			    			$default->log->error("addNewsBL.php DB error inserting dashboard news ($fSynopsis, $fBody, $fRank, with image)");
+			    			$oContent->setHtml(renderAddNewsPage($oDashboardNews, "An error occurred while creating this news item."));
+			    		}	    		
+						
+					} else {
+						// the image is too big
+						$oContent->setHtml(renderAddNewsPage($oDashboardNews, "The image you have submitted is too big (" . $iImgHeight . "x" . $iImgWidth . " > " . $oDashboardNews->getMaxImageDimensions() . "), please correct and retry"));
+					}    			
+				}
+			} else {
+				$default->log->info("no image");
+				// no image uploaded, store what we've got
+				$default->log->info("about to create");
+				if ($oDashboardNews->create()) {
+					// insert worked
+					$default->log->info("addNewsBL.php successfully created dashboard news id=" . $oDashboardNews->getID());
+	    			// redirect to view page
+	    			redirect("$default->rootUrl/control.php?action=viewNews");
+				} else {					
+					// insert failed
+	    			$default->log->error("addNewsBL.php DB error inserting dashboard news ($fSynopsis, $fBody, $fRank, no image)");
+	    			$oContent->setHtml(renderAddNewsPage($oDashboardNews, "An error occurred while creating this news item."));					
+				}
+			}
+	    } else {
+	    	// all params not present, so display an error message
+	    	$oContent->setHtml(renderAddNewsPage($oDashboardNews, "Please complete the form before submitting."));
+	    }
+    } else {
+    	// display the form
+    	$oContent->setHtml(renderAddNewsPage(null));
+    }         
+
+	// build the page
+	require_once("$default->fileSystemRoot/presentation/webpageTemplate.inc");    
+	$main->setCentralPayload($oContent);
+	$main->setFormEncType("multipart/form-data");
+	$main->setFormAction($_SERVER['PHP_SELF']);	
+	$main->setHasRequiredFields(true);			
+	$main->render();	
+} 
+?>
 \ No newline at end of file
@@ -41,21 +41,54 @@ if (checkSession()) {
     		if ($oDashboardNews->getRank() <> $fRank) {
     			$oDashboardNews->setRank($fRank);
     		}
-    		// if we have a new image
-    		// TODO: check that it is an image   		
-    		if (strlen($_FILES['fImage']['name']) > 0) { 
-				$oDashboardNews->setImageFile($_FILES['fImage']['tmp_name']);
-    		}
-    		
-    		if ($oDashboardNews->update()) {
-    			$default->log->info("editNewsBL.php successfully updated dashboard news id=$fNewsID");
-    			// redirect to view page
-    			redirect("$default->rootUrl/control.php?action=viewNews");
+    		// if we have a new image  		
+    		if (strlen($_FILES['fImage']['name']) > 0) {
+    			// return the size of the image
+    			$aSize = getimagesize($_FILES['fImage']['tmp_name']);
+    			// don't accept it if it isn't an image
+    			if (!$aSize) {
+    				$default->log->error("editNewsBL.php attempted to upload a non-image:" . $_FILES['fImage']['name']);
+		    		// display the edit form, with error message    		
+		    		$oContent->setHtml(renderEditNewsPage($oDashboardNews, "You may only upload an image."));
+    				
+    			} else {
+    				// we have an image, now check the size
+    				$iImgWidth = $aSize[0];
+    				$iImgHeight = $aSize[1];
+    				if (DashboardNews::checkImageSize($iImgWidth, $iImgHeight)) {
+    					// size is fine, so set it
+						$oDashboardNews->setImageFile($_FILES['fImage']['tmp_name']);
+
+						// store it
+			    		if ($oDashboardNews->update()) {
+			    			$default->log->info("editNewsBL.php successfully updated dashboard news id=$fNewsID");
+			    			// redirect to view page
+			    			redirect("$default->rootUrl/control.php?action=viewNews");
+			    		} else {
+			    			// update failed
+			    			$default->log->error("editNewsBL.php DB error updating dashboard news id=$fNewsID; ($fSynopsis, $fBody, $fRank)");
+				    		// display the edit form, with error message    		
+				    		$oContent->setHtml(renderEditNewsPage($oDashboardNews, "An error occurred while updating this news item."));
+			    		}    		
+												
+    				} else {
+    					// the image is too big
+			    		$oContent->setHtml(renderEditNewsPage($oDashboardNews, "The image you have submitted is too big (" . $iImgHeight . "x" . $iImgWidth . " > " . $oDashboardNews->getMaxImageDimensions() . "), please correct and retry"));    					
+    				}    			
+    			}
     		} else {
-    			// update failed
-    			$default->log->error("editNewsBL.php DB error updating dashboard news id=$fNewsID; ($fSynopsis, $fBody, $fRank)");
-    			$oContent->setHtml(renderErrorMessage("An error occurred while updating this news item."));
-    		}	    		
+				// store it
+	    		if ($oDashboardNews->update()) {
+	    			$default->log->info("editNewsBL.php successfully updated dashboard news id=$fNewsID");
+	    			// redirect to view page
+	    			redirect("$default->rootUrl/control.php?action=viewNews");
+	    		} else {
+	    			// update failed
+	    			$default->log->error("editNewsBL.php DB error updating dashboard news id=$fNewsID; ($fSynopsis, $fBody, $fRank)");
+	    			$oContent->setHtml(renderErrorMessage("An error occurred while updating this news item."));
+	    		}    		
+    		}    			    		
+    		
     	} else {
     		// display the edit form    		
     		$oContent->setHtml(renderEditNewsPage($oDashboardNews));
@@ -28,12 +28,9 @@ if (checkSession()) {
     	// we have an id, so we're either deleting, editing or previewing a current item
     } else {
     	// no news id, so list the current news items
-    	$oContent->setHtml(renderViewNewsPage());
+    	$oContent->setHtml(renderListNewsPage());
     }        
-	//$main->setFormAction("$default->rootUrl/presentation/lookAndFeel/knowledgeTree/create.php?fRedirectURL=".urlencode("$default->rootUrl/control.php?action=addLinkSuccess&fLinkID="));
-    //$main->setHasRequiredFields(true);
-
 	// build the page
 	require_once("$default->fileSystemRoot/presentation/webpageTemplate.inc");    
 	$main->setCentralPayload($oContent);
@@ -6,28 +6,15 @@ require_once(&quot;$default-&gt;uiDirectory/administration/adminUI.inc&quot;);
 /**
  * $Id$
  *  
- * This page controls browsing for documents- this can be done either by
- * folder, category or document type.
- * The relevant permission checking is performed, calls to the business logic
- * layer to retrieve the details of the documents to view are made and the user
- * interface is contructed.
+ * This page holds all presentation code for administering dashboard news. 
  *
  * Licensed under the GNU GPL. For full terms see the file DOCS/COPYING.
  *
  * @version $Revision$
  * @author Michael Joseph <michael@jamwarehouse.com>, Jam Warehouse (Pty) Ltd, South Africa
- * @package presentation.lookAndFeel.knowledgeTree.documentmanagement
+ * @package presentation.lookAndFeel.knowledgeTree.administration.news
  */
-/**
- * Displays an error message
- */
-function renderErrorMessage($sErrorMessage) {
-	$sToRender  = "<table>";
-	$sToRender .= "<tr><td><p><font color=\"red\">$sErrorMessage</font></p></td></tr>";
-	$sToRender .= "</table>";
-	return $sToRender;
-}
 function renderNewsPopupJavascript() {
 	global $default;
@@ -42,11 +29,21 @@ function renderNewsPopupJavascript() {
 }
 /**
+ * Displays an error message
+ */
+function renderErrorMessage($sErrorMessage) {
+	$sToRender  = "<table>";
+	$sToRender .= "<tr><td><p><font color=\"red\">$sErrorMessage</font></p></td></tr>";
+	$sToRender .= "</table>";
+	return $sToRender;
+}
+
+/**
  * Displays the current news items
  */	
 function renderNewsTable() {
     global $default;
-    $sQuery = "SELECT id, synopsis, body, rank, if((length(image)>0),'yes','no') as image, 'Edit' AS edit, 'Preview' AS preview, 'Delete' AS del " .   
+    $sQuery = "SELECT id, synopsis, IF(LENGTH(body) > 25, CONCAT(LEFT(body,25),'...'), body) AS body, rank, IF(LENGTH(image)>0, 'yes', 'no') as image, 'Edit' AS edit, 'Preview' AS preview, 'Delete' AS del " .   
               "FROM $default->owl_news_table ORDER BY rank ASC";
     $aColumns = array("synopsis", "body", "rank", "image", "edit", "preview", "del");
     $aColumnHeaders = array("Synopsis", "Body", "Rank", "Image");
@@ -57,65 +54,78 @@ function renderNewsTable() {
                        5=> "$default->rootUrl/control.php?action=previewNews",
                        6=> "$default->rootUrl/control.php?action=removeNews");
-    $oPatternTableSqlQuery = & new PatternTableSqlQuery($sQuery, $aColumns, $aColumnTypes, $aColumnHeaders, "10%", $aLinkURLs,$aDBColumnArray,$aQueryStringVariableNames);
+    $oPatternTableSqlQuery = & new PatternTableSqlQuery($sQuery, $aColumns, $aColumnTypes, $aColumnHeaders, "100%", $aLinkURLs,$aDBColumnArray,$aQueryStringVariableNames);
     $oPatternTableSqlQuery->setTableHeading("Dashboard News");
     $oPatternTableSqlQuery->setDisplayColumnHeadings(true);        
     return $oPatternTableSqlQuery->render();
 }
 /**
+ * Displays a news item
+ */
+function renderNewsItem($oDashboardNews) {
+	global $default;
+	
+	$sToRender .= "<tr><td>Synopsis:</td><td>" . $oDashboardNews->getSynopsis() . "</td></tr>";
+	$sToRender .= "<tr><td>Body:</td><td>" . $oDashboardNews->getBody() . "</td></tr>";
+	$sToRender .= "<tr><td>Rank:</td><td>" . $oDashboardNews->getRank() . "</td></tr>";	
+	$sToRender .= "<tr><td colspan=\"2\"><img src=\"$default->rootUrl/" . $default->siteMap->getPage("viewNewsImage") . "?fNewsID=" . $oDashboardNews->getID() . "\" border=\"0\"></td></tr>";	
+	$sToRender .= "<input type=\"hidden\" name=\"fNewsID\" value=\"" . $oDashboardNews->getID() . "\">";
+    return $sToRender;
+}
+
+/**
  * Displays the edit/add news item form
  *
  * @param DashboardNews the news item to modify
  */
-function renderModifyAddNewsForm($oDashboardNews = null) {
+function renderEditAddNewsForm($oDashboardNews, $bUpdate) {
 	global $default;
 	$sToRender  = "<table>";
-	$sToRender .= "<tr><td>Synopsis</td><td><input type=\"text\" name=\"fSynopsis\" value=\"" . ($oDashboardNews ? $oDashboardNews->getSynopsis() : "") . "\"></td></tr>";
-	$sToRender .= "<tr><td>Body</td><td><input type=\"textarea\" name=\"fBody\" value=\"" . ($oDashboardNews ? $oDashboardNews->getBody() : "") . "\"></td></tr>";
-	$sToRender .= "<tr><td>Rank</td><td><input type=\"text\" name=\"fRank\" value=\"" . ($oDashboardNews ? $oDashboardNews->getRank() : "") . "\"></td></tr>";	
-	$sToRender .= "<tr><td>Image</td><td><input type=\"file\" name=\"fImage\"></td></tr>";
-	$sToRender .= (isset($oDashboardNews) ? "<input type=\"hidden\" name=\"fNewsID\" value=\"" . $oDashboardNews->getID() . "\">" : "");
-	$sToRender .= "<input type=\"hidden\" name=\"fUpdate\" value=\"1\">";	
+	
+	$sToRender .= "<tr><td>Synopsis:</td><td><input type=\"text\" name=\"fSynopsis\" value=\"" . ($oDashboardNews ? $oDashboardNews->getSynopsis() : "") . "\"></td></tr>";
+	$sToRender .= "<tr><td>Body:</td><td><textarea name=\"fBody\" cols=\"35\" rows=\"10\">" . ($oDashboardNews ? $oDashboardNews->getBody() : "") . "</textarea></td></tr>";
+	$sToRender .= "<tr><td>Rank:</td><td><input type=\"text\" name=\"fRank\" value=\"" . ($oDashboardNews ? $oDashboardNews->getRank() : "") . "\"></td></tr>";	
+	$sToRender .= "<tr><td>Image:</td><td><input type=\"file\" name=\"fImage\"></td></tr>";
+	
+	if ($bUpdate) {
+		$sToRender .= "<input type=\"hidden\" name=\"fNewsID\" value=\"" . $oDashboardNews->getID() . "\">";
+		$sToRender .= "<input type=\"hidden\" name=\"fUpdate\" value=\"1\">";		
+	} else {
+		$sToRender .= "<input type=\"hidden\" name=\"fStore\" value=\"1\">";
+	}	
+			
 	$sToRender .= "<tr><td><input type=\"image\" src=\"$default->graphicsUrl/widgets/submit.gif\" border=\"0\">\n";
 	$sToRender .= "<a href=\"$default->rootUrl/control.php?action=viewNews\"><img src=\"$default->graphicsUrl/widgets/cancel.gif\" border=\"0\"></a></td></tr>\n";    
 	$sToRender .= "</table>";
+	
+	$sToRender .= "\n\n<script language=\"javascript\">\n<!--\n";
+	$sToRender .= "function validateForm(theForm) {\n";	
+	$sToRender .= "\tif (!(validRequired(theForm.fSynopsis,'Synopsis'))) {\n";
+	$sToRender .= "\t\treturn false;\n\t}\n";
+    $sToRender .= "\tif (!(validRequired(theForm.fBody,'Body'))) {\n";
+	$sToRender .= "\t\treturn false;\n\t}\n";
+	$sToRender .= "\tif (!(validNum(theForm.fRank,'Rank')) || !(validRequired(theForm.fRank,'Rank'))) {\n";
+	$sToRender .= "\t\treturn false;\n\t}\n";	
+	$sToRender .= "return true;\n}\n";
+	$sToRender .= "//-->\n</script>\n\n";
+		
 	return $sToRender; 
 } 
-
-/**
- * Prompts for confirmation when deleting a news item
- */
-function renderDeleteNewsConfirmationPage($oDashboardNews) {
-	global $default;
-	
-	$sToRender .= "<table width=\"600\">" . renderHeading("Delete Dashboard News") . "</table>";
-    $sToRender .= "<table>\n";
-	$sToRender .= "<tr><td>Are you sure you wish to delete this news item?</td></tr>";
-	$sToRender .= "<tr><td>Synopsis</td><td>" . $oDashboardNews->getSynopsis() . "</td></tr>";
-	$sToRender .= "<tr><td>Body</td><td>" . $oDashboardNews->getBody() . "</td></tr>";
-	$sToRender .= "<tr><td>Rank</td><td>" . $oDashboardNews->getRank() . "</td></tr>";	
-	$sToRender .= "<tr><td colspan=\"2\"><img src=\"$default->rootUrl/" . $default->siteMap->getPage("viewNewsImage") . "?fNewsID=" . $oDashboardNews->getID() . "\" border=\"0\"></td></tr>";
-	$sToRender .= "<input type=\"hidden\" name=\"fNewsID\" value=\"" . $oDashboardNews->getID() . "\">";
-	$sToRender .= "<input type=\"hidden\" name=\"fDeleteConfirmed\" value=\"1\">";	
-	$sToRender .= "<tr><td><input type=\"image\" src=\"$default->graphicsUrl/widgets/delete.gif\" border=\"0\">\n";
-	$sToRender .= "<a href=\"$default->rootUrl/control.php?action=viewNews\"><img src=\"$default->graphicsUrl/widgets/cancel.gif\" border=\"0\"></a></td></tr>\n";    
-	
-	$sToRender .= "</table>\n";
-	return $sToRender;	
-}
-
 /**
  * Displays the edit news page
  */
-function renderEditNewsPage($oDashboardNews) {
+function renderEditNewsPage($oDashboardNews, $sErrorMessage = "") {
 	global $default;
 	$sToRender .= "<table width=\"600\">" . renderHeading("Edit Dashboard News") . "</table>";
+	if (strlen($sErrorMessage) > 0) {
+		$sToRender .= renderErrorMessage($sErrorMessage);
+	}
     $sToRender .= "<table>\n";
-	$sToRender .= renderModifyAddNewsForm($oDashboardNews);
+	$sToRender .= renderEditAddNewsForm($oDashboardNews, true);
 	$sToRender .= "</table>\n";
 	return $sToRender;
 }
@@ -123,19 +133,22 @@ function renderEditNewsPage($oDashboardNews) {
 /**
  * Displays the add new new item page
  */
-function renderNewNewsPage() {
+function renderAddNewsPage($oDashboardNews, $sErrorMessage = "") {
 	global $default;
 	$sToRender .= "<table width=\"600\">" . renderHeading("View Dashboard News") . "</table>";
+	if (strlen($sErrorMessage) > 0) {
+		$sToRender .= renderErrorMessage($sErrorMessage);
+	}	
     $sToRender .= "<table>\n";
-	$sToRender .= renderModifyAddNewsForm();
+	$sToRender .= renderEditAddNewsForm($oDashboardNews, false);
 	$sToRender .= "</table>\n";
 	return $sToRender;	
 }
 /**
  * Displays a list of news items
  */
-function renderViewNewsPage() {
+function renderListNewsPage() {
 	global $default;
 	$sToRender .= "<table width=\"600\">" . renderHeading("View Dashboard News") . "</table>";
@@ -143,5 +156,39 @@ function renderViewNewsPage() {
 	$sToRender .= renderNewsTable();
 	$sToRender .= "</table>\n";
 	return $sToRender;
+}
+
+/**
+ * Displays the preview page
+ */
+function renderPreviewNewsPage($oDashboardNews) {
+	global $default;
+	
+	$sToRender .= "<table width=\"600\">" . renderHeading("Preview Dashboard News") . "</table>";
+    $sToRender .= "<table>\n";	
+	$sToRender .= renderNewsItem($oDashboardNews);		
+	$sToRender .= "<tr><td><a href=\"$default->rootUrl/control.php?action=viewNews\"><img src=\"$default->graphicsUrl/widgets/back.gif\" border=\"0\"></a></td></tr>\n";    	
+	$sToRender .= "</table>\n";
+	return $sToRender;		
 } 
+
+/**
+ * Prompts for confirmation when deleting a news item
+ */
+function renderDeleteNewsConfirmationPage($oDashboardNews) {
+	global $default;
+	
+	$sToRender .= "<table width=\"600\">" . renderHeading("Delete Dashboard News") . "</table>";
+    $sToRender .= "<table>\n";
+	$sToRender .= "<tr><td colspan=\"2\">Are you sure you wish to delete this news item?</td></tr>";
+	
+	$sToRender .= renderNewsItem($oDashboardNews);	
+	
+	$sToRender .= "<input type=\"hidden\" name=\"fDeleteConfirmed\" value=\"1\">";	
+	$sToRender .= "<tr><td><input type=\"image\" src=\"$default->graphicsUrl/widgets/delete.gif\" border=\"0\">\n";
+	$sToRender .= "<a href=\"$default->rootUrl/control.php?action=viewNews\"><img src=\"$default->graphicsUrl/widgets/cancel.gif\" border=\"0\"></a></td></tr>\n";    
+	
+	$sToRender .= "</table>\n";
+	return $sToRender;	
+}
 ?>
 \ No newline at end of file
+<?php
+
+require_once("../../../../../config/dmsDefaults.php");
+require_once("$default->fileSystemRoot/lib/dashboard/DashboardNews.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternMainPage.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternCustom.inc");
+require_once("$default->fileSystemRoot/lib/visualpatterns/PatternTableSqlQuery.inc");
+require_once("$default->uiDirectory/administration/news/newsUI.inc");
+
+/**
+ * $Id$
+ *  
+ * This page previews a news item
+ *
+ * Licensed under the GNU GPL. For full terms see the file DOCS/COPYING.
+ *
+ * @version $Revision$
+ * @author Michael Joseph <michael@jamwarehouse.com>, Jam Warehouse (Pty) Ltd, South Africa
+ * @package presentation.lookAndFeel.knowledgeTree.administration.news
+ */
+
+if (checkSession()) {	
+		
+    // instantiate my content pattern
+    $oContent = new PatternCustom();
+    
+    if (isset($fNewsID)) {
+    	// we have an id, so we're can proceed
+    	$oContent->setHtml(renderPreviewNewsPage(DashboardNews::get($fNewsID)));
+    } else {
+    	// no news id, nothing to preview
+    	$oContent->setHtml(renderListNewsPage());
+    }        
+
+	// build the page
+	require_once("$default->fileSystemRoot/presentation/webpageTemplate.inc");    
+	$main->setCentralPayload($oContent);
+	$main->render();	
+} 
+?>
 \ No newline at end of file
@@ -31,13 +31,14 @@ if (checkSession()) {
     	if ($fDeleteConfirmed) {
     		// confirmation received, so delete    		
     		if ($oDashboardNews->delete()) {
-    			$default->log->info("deleteNewsBL.php successfully delete dashboard news id=$fNewsID");
+    			$default->log->info("deleteNewsBL.php successfully deleted dashboard news id=$fNewsID");
     			// redirect to view page
     			redirect("$default->rootUrl/control.php?action=viewNews");
     		} else {
     			// delete failed
     			$default->log->error("deleteNewsBL.php DB error deleting dashboard news (" . arrayToString($oDashboardNews) . ")");
     			$oContent->setHtml(renderErrorMessage("An error occurred while deleting this news item."));
+    			// TODO: incorporate message into another page
     		}	    		
     	} else {
     		// display the delete confirmation page		
@@ -46,6 +47,7 @@ if (checkSession()) {
     } else {
     	// no news id, so display an error message
     	$oContent->setHtml(renderErrorMessage("No news item was selected for deletion"));
+    	// TODO: incorporate message into another page
     }        
 	// build the page