Merge pull request #17 from ThatGerber/hotfix

Fixed issue where settings were not being called.

Author: chriswgerber

includes/helper_functions.php

@@ -231,7 +231,7 @@ function dfp_get_url() {
  * @return array|string|int|bool
  */
 function dfp_get_settings_value( $setting ) {
-	$option_array = get_option( 'DFP_Ad_Positions' );
+	$option_array = get_option( 'DFP_Ads_Settings' );
 
 	return $option_array[$setting];
 }

plugin.php

@@ -12,31 +12,31 @@
  * License URI:       http://www.gnu.org/licenses/gpl-2.0.txt
  * Text Domain:       dfp-ads
  * Github Plugin URI: https://github.com/ThatGerber/dfp-ads
- * GitHub Branch:     master
- * Version:           0.2.0
+ * GitHub Branch:     stable
+ * Version:           0.2.1
  *
  * The Plugin File
  *
  * @link              http://www.chriswgerber.com/dfp-ads
  * @since             0.0.1
  * @subpackage        DFP-Ads
  */
-define( 'EPG_AD_PLUGIN_VER', '0.2.0' );
+define( 'EPG_AD_PLUGIN_VER', '0.2.1' );
 
 /* Autoload */
 require_once 'vendor/autoload.php';
 
 /* Library */
-include( 'includes/helper_functions.php' );
-include( 'includes/abstract.dfp_ads_form.php' );
-include( 'includes/class.dfp_ads.php' );
-include( 'includes/class.dfp_ads_post_type.php' );
-include( 'includes/class.dfp_ads_input.php' );
-include( 'includes/class.dfp_ad_position.php' );
-include( 'includes/class.dfp_ads_settings_form.php' );
-include( 'includes/class.dfp_ads_import_form.php' );
-include( 'includes/class.dfp_ads_admin.php' );
-include( 'widget/widget.ad_position.php' );
+include 'includes/helper_functions.php';
+include 'includes/abstract.dfp_ads_form.php';
+include 'includes/class.dfp_ads.php';
+include 'includes/class.dfp_ads_post_type.php';
+include 'includes/class.dfp_ads_input.php';
+include 'includes/class.dfp_ad_position.php';
+include 'includes/class.dfp_ads_settings_form.php';
+include 'includes/class.dfp_ads_import_form.php';
+include 'includes/class.dfp_ads_admin.php';
+include 'widget/widget.ad_position.php';
 
 /*
  * Initialization for Post Type
@@ -62,13 +62,13 @@
 /* Begin creating the new ads objects */
 $dfp_ads             = new DFP_Ads();
 $dfp_ads->dir_uri    = plugins_url( null, __FILE__ );
-$dfp_ads->set_account_id( dfp_get_settings_value( 'dfp_property_code' ) ); // = '/35190362/';
+$dfp_ads->set_account_id( dfp_get_settings_value( 'dfp_property_code' ) );
 
 /*
  * Enqueues the styles and scripts into WordPress. When this action runs
  * it also will grab all of the positions and other filtered in information
  */
-add_action( 'wp_enqueue_scripts', array($dfp_ads, 'scripts_and_styles') );
+add_action( 'wp_enqueue_scripts', array( $dfp_ads, 'scripts_and_styles' ) );
 
 /* Sets Menu Position. Default 20 */
 add_filter( 'dfp_ads_menu_position', ( function( $pos ) { return 79; }), 10 );

readme.md

@@ -85,6 +85,10 @@ Submit your questions at [chriswgerber.com/contact/](http://www.chriswgerber.com
 <a name="Changelog"></a>
 ## Changelog
 
+### 0.2.1
+
+* Fixed issue where settings were not being called.
+
 ### 0.2.0
 
 * Added functionality for importing DFP Ads

readme.txt

@@ -1,6 +1,6 @@
 === DFP Ad Manager ===
 
-Stable tag: trunk
+Stable tag: 0.2.1
 Contributors: chriswgerber
 Requires at least: 3.0.0
 Tested up to: 4.2.2
@@ -56,6 +56,10 @@ Submit your questions at [chriswgerber.com/contact/](http://www.chriswgerber.com
 
 == Changelog ==
 
+= 0.2.1
+
+* Fixed issue where settings were not being called.
+
 = 0.2.0
 
 * Added functionality for importing DFP Ads