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.
|
