Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Commit

Permalink
More validation for update and add
Browse files Browse the repository at this point in the history
  • Loading branch information
alexhambley committed Apr 24, 2018
1 parent d5718c4 commit 9b4daf6
Show file tree
Hide file tree
Showing 6 changed files with 46 additions and 37 deletions.
4 changes: 2 additions & 2 deletions src/_updateRole.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
var_dump($_POST);
$elements = implode(",", array_filter($_POST['elements']));
$themes = implode(",", array_filter($_POST['themes']));
$tempAlternativeNames = array($_POST['altName1'], $_POST['altName2'], $_POST['altName3'], $_POST['altName4'], $_POST['altName5']);
$names = implode(",", array_filter($tempAlternativeNames));
$names = implode(",", array_filter($_POST['altName']));
// $names = implode(",", array_filter($tempAlternativeNames));

if (empty($_POST['entry']) || empty($_POST['newEntry']) || empty($_POST['description'])) {
header( "refresh:3;url=index_admin.php" );
Expand Down
4 changes: 2 additions & 2 deletions src/admin_add_elements.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class="btn btn-default"
</div>
</div>
<div class="text-center">
<div class="btn-group btn-group-lg" role="group" aria-label="Basic example">
<div class="btn-group btn-group-lg" role="group">
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_add_roles.php'">
Expand All @@ -73,7 +73,7 @@ class="btn btn-secondary"
<h2> Add Elements </h2>
</div>
<p> Please use this form to insert new elements to the database. </p>

<div class="form-group">
<label for="elename"> New element name: </label>
<input type="text" class="form-control" name="name" placeholder="New element name" required>
Expand Down
2 changes: 1 addition & 1 deletion src/admin_add_roles.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class="btn btn-default"
</div>

<div class="text-center">
<div class="btn-group btn-group-lg" role="group" aria-label="Basic example">
<div class="btn-group btn-group-lg" role="group">
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_add_roles.php'"
Expand Down
8 changes: 5 additions & 3 deletions src/admin_add_themes.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class="btn btn-default"
</div>

<div class="text-center">
<div class="btn-group btn-group-lg" role="group" aria-label="Basic example">
<div class="btn-group btn-group-lg" role="group">
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_add_roles.php'">
Expand Down Expand Up @@ -76,12 +76,12 @@ class="btn btn-secondary"

<div class="form-group">
<label for="themename"> New theme name: </label>
<input type="text" class="form-control" name="name" placeholder="New theme name">
<input type="text" class="form-control" name="name" placeholder="New theme name" required>
</div>

<div class="form-group">
<label for="themedesc"> New theme explanation: </label>
<textarea class="form-control" name="explanation" rows="1"></textarea>
<textarea class="form-control" name="explanation" rows="1" required></textarea>
</div>

<div class="form-group">
Expand All @@ -99,6 +99,8 @@ class="btn btn-secondary"
echo "<br>";
}
?>
<small id="rehelp" class="form-text text-muted">Must select at least one</small>

</div>
<input type="submit" class="btn btn-success" style="background-color: #2a8c3e" value="Add to database">
</form>
Expand Down
34 changes: 19 additions & 15 deletions src/admin_update_roles.php
View file
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
<?php
session_start();

include "header.php";
include "navbar.php";
include "db.php";
?>
<script>
var altNameLimit = 5;
var currNameNumber = 1;

function myFunc(numOfNames) {
if (altNameLimit == currNameNumber) {
alert ("You can't add any more alternative names")
Expand All @@ -31,15 +31,15 @@ function myFunc(numOfNames) {
<div class="container">
<div class="row">
<div class="col-sm-4">
<div style="padding-top: 30px;">
<button type="button"
<div style="padding-top: 30px;">
<button type="button"
class="btn btn-default"
onclick="window.location.href='index_admin.php'">
<span class="glyphicon glyphicon-arrow-left"> </span>
<span class="glyphicon glyphicon-arrow-left"> </span>
Back to Admin Page
</button>
</button>
</div>

</div>
<div class="col-sm-4">
<div class="text-center">
Expand All @@ -50,26 +50,26 @@ class="btn btn-default"

<div class="text-center">
<div class="btn-group btn-group-lg" role="group">
<button type="button"
class="btn btn-secondary"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_roles.php'"
disabled>
disabled>
&nbsp&nbspRoles&nbsp
</button>
<button type="button"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_themes.php'">
&nbspThemes&nbsp
</button>
<button type="button"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_elements.php'">
Elements
</button>
</div>
</div>



<!-- Update Role -->
<form action="_updateRole.php" method="post">
Expand All @@ -96,18 +96,18 @@ class="btn btn-secondary"
<br>
<div class="form-group">
<label for="themename"> Updated role name: </label>
<input type="text" class="form-control" name="newEntry" placeholder="New role entry name">
<input type="text" class="form-control" name="newEntry" placeholder="New role entry name" required>
</div>
<div class="form-group">
<label for="roledesc"> Updated role description: </label>
<textarea class="form-control" name="description" rows="2"></textarea>
<textarea class="form-control" name="description" rows="2" required></textarea>
</div>
<div class="form-group">
<label for="rolealt"> Alternative Names: </label>
<div id="numOfNames">
Name 1
<br>
<input type="text" class="form-control" classname="altName[]">
<input type="text" class="form-control" classname="altName[]" required>
</div>
<br>
<button type=button class="btn btn-default" onclick="return myFunc('numOfNames')"> Add another name </button>
Expand All @@ -128,6 +128,8 @@ class="btn btn-secondary"
echo "<br>";
}
?>
<small id="rehelp" class="form-text text-muted">Must select at least one</small>

</div>
<div class="form-group">
<label for="rolethemes"> Related Themes: </label>
Expand All @@ -144,6 +146,8 @@ class="btn btn-secondary"
echo "<br>";
}
?>
<small id="rehelp" class="form-text text-muted">Must select at least one</small>

</div>
<input type="submit" class="btn btn-success" style="background-color: #2a8c3e" value="Update the database">
</form>
Expand Down
31 changes: 17 additions & 14 deletions src/admin_update_themes.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,15 @@ function myFunc(numOfNames) {
<div class="container">
<div class="row">
<div class="col-sm-4">
<div style="padding-top: 30px;">
<button type="button"
<div style="padding-top: 30px;">
<button type="button"
class="btn btn-default"
onclick="window.location.href='index_admin.php'">
<span class="glyphicon glyphicon-arrow-left"> </span>
<span class="glyphicon glyphicon-arrow-left"> </span>
Back to Admin Page
</button>
</button>
</div>

</div>
<div class="col-sm-4">
<div class="text-center">
Expand All @@ -50,18 +50,18 @@ class="btn btn-default"
<div class="text-center">

<div class="btn-group btn-group-lg" role="group">
<button type="button"
class="btn btn-secondary"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_roles.php'">
&nbsp&nbspRoles&nbsp
</button>
<button type="button"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_themes.php'"
disabled>
&nbspThemes&nbsp
</button>
<button type="button"
<button type="button"
class="btn btn-secondary"
onclick="window.location.href='admin_update_elements.php'">
Elements
Expand Down Expand Up @@ -94,8 +94,14 @@ class="btn btn-secondary"

<div class="form-group">
<label for="themename"> Updated theme name: </label>
<input type="text" class="form-control" name="newName" placeholder="New theme name">
<input type="text" class="form-control" name="newName" placeholder="New theme name" required>
</div>

<div class="form-group">
<label for="themedesc"> New theme explanation: </label>
<textarea class="form-control" name="explanation" rows="1" required></textarea>
</div>

<div class="form-group">
<label for="themeelements"> Related Elements: </label>
<br>
Expand All @@ -111,10 +117,7 @@ class="btn btn-secondary"
echo "<br>";
}
?>
</div>
<div class="form-group">
<label for="themedesc"> New theme explanation: </label>
<textarea class="form-control" name="explanation" rows="1"></textarea>
<small id="rehelp" class="form-text text-muted">Must select at least one</small>
</div>
<input type="submit" class="btn btn-success" style="background-color: #2a8c3e" value="Update the database">
</form>
Expand Down

0 comments on commit 9b4daf6

Please sign in to comment.