fixed wrong error message in schema assignment

This commit is contained in:
Andreas Gohr
2016-02-24 16:20:01 +01:00
parent fd81b928af
commit 28e054687d

View File

@ -45,19 +45,25 @@ class admin_plugin_struct_assignments extends DokuWiki_Admin_Plugin {
*/ */
public function handle() { public function handle() {
global $INPUT; global $INPUT;
global $ID;
$assignments = new Assignments(); $assignments = new Assignments();
if($INPUT->str('action') && $INPUT->arr('assignment') && checkSecurityToken()) { if($INPUT->str('action') && $INPUT->arr('assignment') && checkSecurityToken()) {
$assignment = $INPUT->arr('assignment'); $assignment = $INPUT->arr('assignment');
$ok = false; $ok = true;
if ($INPUT->str('action') === 'delete') { if(!blank($assignment['assign']) && !blank($assignment['tbl'])) {
$ok = $assignments->remove($assignment['assign'], $assignment['tbl']); if($INPUT->str('action') === 'delete') {
} else if($INPUT->str('action') === 'add') { $ok = $assignments->remove($assignment['assign'], $assignment['tbl']);
$ok = $assignments->add($assignment['assign'], $assignment['tbl']); } else if($INPUT->str('action') === 'add') {
$ok = $assignments->add($assignment['assign'], $assignment['tbl']);
}
} }
if(empty($sql) || empty($assignment['assign']) || empty($assignment['tbl']) || !$ok) {
if(!$ok) {
msg('something went wrong while saving', -1); msg('something went wrong while saving', -1);
} }
send_redirect(wl($ID, array('do' => 'admin', 'page' => 'struct_assignments'), true, '&'));
} }
} }
@ -80,12 +86,10 @@ class admin_plugin_struct_assignments extends DokuWiki_Admin_Plugin {
$ass = new Assignments(); $ass = new Assignments();
$assignments = $ass->getAll(); $assignments = $ass->getAll();
echo '<form action="' . wl($ID) . '" action="post">';
echo '<form action="'.wl($ID).'">';
echo '<input type="hidden" name="do" value="admin" />'; echo '<input type="hidden" name="do" value="admin" />';
echo '<input type="hidden" name="page" value="struct_assignments" />'; echo '<input type="hidden" name="page" value="struct_assignments" />';
echo '<input type="hidden" name="sectok" value="'.getSecurityToken().'" />'; echo '<input type="hidden" name="sectok" value="' . getSecurityToken() . '" />';
echo '<table class="inline">'; echo '<table class="inline">';
// header // header
@ -96,23 +100,25 @@ class admin_plugin_struct_assignments extends DokuWiki_Admin_Plugin {
echo '</tr>'; echo '</tr>';
// existing assignments // existing assignments
foreach ($assignments as $assignment) { foreach($assignments as $assignment) {
$schema = $assignment['tbl']; $schema = $assignment['tbl'];
$assignee = $assignment['assign']; $assignee = $assignment['assign'];
$link = wl($ID, array( $link = wl(
$ID, array(
'do' => 'admin', 'do' => 'admin',
'page' => 'struct_assignments', 'page' => 'struct_assignments',
'action' => 'delete', 'action' => 'delete',
'sectok' => getSecurityToken(), 'sectok' => getSecurityToken(),
'assignment[tbl]' => $schema, 'assignment[tbl]' => $schema,
'assignment[assign]' => $assignee, 'assignment[assign]' => $assignee,
)); )
);
echo '<tr>'; echo '<tr>';
echo '<td>'.hsc($assignee).'</td>'; echo '<td>' . hsc($assignee) . '</td>';
echo '<td>'.hsc($schema).'</td>'; echo '<td>' . hsc($schema) . '</td>';
echo '<td><a href="'.$link.'">Delete</a></td>'; //FIXME localize echo '<td><a href="' . $link . '">Delete</a></td>'; //FIXME localize
echo '</tr>'; echo '</tr>';
} }
@ -121,8 +127,8 @@ class admin_plugin_struct_assignments extends DokuWiki_Admin_Plugin {
echo '<td><input type="text" name="assignment[assign]" /></td>'; echo '<td><input type="text" name="assignment[assign]" /></td>';
echo '<td>'; echo '<td>';
echo '<select name="assignment[tbl]">'; echo '<select name="assignment[tbl]">';
foreach ($schemas as $schema){ foreach($schemas as $schema) {
echo '<option value="'. hsc($schema['tbl']) .'">'. hsc($schema['tbl']) . '</option>'; echo '<option value="' . hsc($schema['tbl']) . '">' . hsc($schema['tbl']) . '</option>';
} }
echo '</select>'; echo '</select>';
echo '</td>'; echo '</td>';