[xoops-svn] SF.net SVN: xoops:[8681] XoopsModules/xrest/releases/1.52/htdocs/modules/ xrest

[<wis...@us...>]

Revision: 8681
          http://xoops.svn.sourceforge.net/xoops/?rev=8681&view=rev
Author:   wishcraft
Date:     2012-01-09 00:33:56 +0000 (Mon, 09 Jan 2012)
Log Message:
-----------
X-REST 1.52 - Rest API for XOOPS 2.5 - Usability Issue with Admin Fields and No Fields for tables added to database.

Modified Paths:
--------------
    XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/include/forms.xrest.php
    XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/language/english/admin.php

Modified: XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/include/forms.xrest.php
===================================================================
--- XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/include/forms.xrest.php	2012-01-09 00:32:32 UTC (rev 8680)
+++ XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/include/forms.xrest.php	2012-01-09 00:33:56 UTC (rev 8681)
@@ -27,6 +27,11 @@
 		$fields_handler = xoops_getmodulehandler('fields', 'xrest');
 		
 		$table = $tables_handler->get($tbl_id);
+		
+		if (!is_object($table)) {
+			redirect_header(XOOPS_URL.'/modules/xrest/admin/index.php?op=tables', 10, _XREST_AM_MSG_NEEDTOSAVETABLES_FIRST);
+			exit;
+		}
 
 		$fields = $fields_handler->getFieldFromTable($table->getVar('tablename'));
 		

Modified: XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/language/english/admin.php
===================================================================
--- XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/language/english/admin.php	2012-01-09 00:32:32 UTC (rev 8680)
+++ XoopsModules/xrest/releases/1.52/htdocs/modules/xrest/language/english/admin.php	2012-01-09 00:33:56 UTC (rev 8681)
@@ -5,7 +5,7 @@
 	define('_XREST_AM_MSG_SAVETABLES_DATABASE_UPDATED', 'Saving Tables is complete, database has been updated!');
 	define('_XREST_AM_MSG_SAVEVIEWS_DATABASE_UPDATED', 'Saving Views is complete, database has been updated!');
 	define('_XREST_AM_MSG_SAVEPLUGINS_DATABASE_UPDATED', 'Saving Plugins is complete, database has been updated - Don\'t forget to set your permissions!');
-	
+	define('_XREST_AM_MSG_NEEDTOSAVETABLES_FIRST', 'Either the table ID you have selected doesn\'t exist or you haven\'t saved any tables in the database yet!');
 	//About box
 	define('_XREST_AM_MAKE_DONATION', 'Make XRest Even Better - Make a donation now!');
 	

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.