Skip to content

Commit

Permalink
Fix Issue #6 required fields
Browse files Browse the repository at this point in the history
  • Loading branch information
leocaseiro committed Feb 4, 2016
1 parent a80c61e commit 45b4a28
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 7 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,10 @@ FAQ (Frequently Asked Questions)
Changelog
--------------

**1.6**

* Set all fields as required [Fix Issue #6](https://github.com/leocaseiro/Wordpress-Plugin-Custom-Options-Plus/issues/6)

**1.5**

* Lot of best practices improvements on code
Expand Down
8 changes: 4 additions & 4 deletions custom-options-plus.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
Description: The easiest way to add your custom variables as a Settings Page for your Theme. Even with no expertise in PHP.
You can for example, register the address and phone numbers of your company to leave in the header of your site. So, if someday relocate, you do not need to change your theme. Just change administratively.
You can also enter the login of your social networks. How to login twitter, Facebook, Youtube, contact email and more.
Version: 1.5
Version: 1.6
Author: Leo Caseiro
Author URI: http://leocaseiro.com.br/
*/
Expand Down Expand Up @@ -253,23 +253,23 @@ function custom_options_plus_adm() {
<label for="label">Label:</label>
</td>
<td>
<input name="label" type="text" id="label" value="<?php echo $label; ?>" class="regular-text">
<input name="label" required="required" type="text" id="label" value="<?php echo $label; ?>" class="regular-text">
</td>
</tr>
<tr>
<th scope="row">
<label for="name">*Name:</label>
</td>
<td>
<input name="name" type="text" id="name" value="<?php echo $name; ?>" class="regular-text">
<input name="name" required="required" type="text" id="name" value="<?php echo $name; ?>" class="regular-text">
</td>
</tr>
<tr>
<th scope="row">
<label for="value">Value:</label>
</td>
<td>
<textarea name="value" rows="7" cols="40" type="text" id="value" class="regular-text code"><?php echo $value; ?></textarea>
<textarea required="required" name="value" rows="7" cols="40" type="text" id="value" class="regular-text code"><?php echo $value; ?></textarea>
</td>
</tr>
</tbody>
Expand Down
10 changes: 7 additions & 3 deletions readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ Contributors: leocaseiro
Donate link: http://leocaseiro.com.br/contato/
Tags: configs, custom, custom configs, custom options, custom options plus, custom settings, leocaseiro, options, settings, wp_options
Requires at least: 2.7
Tested up to: 4.2
Stable tag: 1.5
Tested up to: 4.4.2
Stable tag: 1.6
License: GPLv2 or later
License URI: http://www.gnu.org/licenses/gpl-2.0.html

Expand Down Expand Up @@ -45,6 +45,10 @@ This Plugin was Based on Custom Settings (Custom Configs) which has been removal

== Changelog ==

= 1.6 =

* Set all fields as required [Fix Issue #6](https://github.com/leocaseiro/Wordpress-Plugin-Custom-Options-Plus/issues/6)

= 1.5 =

* Lot of best practices improvements on code
Expand All @@ -66,7 +70,7 @@ This Plugin was Based on Custom Settings (Custom Configs) which has been removal

= 1.1 =
* Value field from varchar(255) to text
* SQL Injection fix following suggestion from Andy Stratton in http://wordpress.org/support/topic/plugin-custom-options-plus-stripslashes-needed-on-submission-of-content?replies=1
* [SQL Injection fix following suggestion from Andy Stratton](http://wordpress.org/support/topic/plugin-custom-options-plus-stripslashes-needed-on-submission-of-content?replies=1)
* New Layout using WP List Table


Expand Down

0 comments on commit 45b4a28

Please sign in to comment.