test import from url #141

Author: contactashish13

classes/Visualizer/Module/Chart.php

@@ -443,21 +443,22 @@ public function renderFlattrScript() {
 	 */
 	public function uploadData() {
 		// validate nonce
-		if ( ! wp_verify_nonce( filter_input( INPUT_GET, 'nonce' ) ) ) {
+		// do not use filter_input as it does not work for phpunit test cases, use filter_var instead
+		if ( ! isset( $_GET['nonce'] ) || ! wp_verify_nonce( $_GET['nonce'] ) ) {
 			status_header( 403 );
 			exit;
 		}
 
 		// check chart, if chart exists
-		$chart_id = filter_input( INPUT_GET, 'chart', FILTER_VALIDATE_INT );
+		$chart_id = isset( $_GET['chart'] ) ? filter_var( $_GET['chart'], FILTER_VALIDATE_INT ) : '';
 		if ( ! $chart_id || ! ( $chart = get_post( $chart_id ) ) || $chart->post_type != Visualizer_Plugin::CPT_VISUALIZER ) {
 			status_header( 400 );
 			exit;
 		}
 
 		$source = null;
 		$render = new Visualizer_Render_Page_Update();
-		if ( filter_input( INPUT_POST, 'remote_data', FILTER_VALIDATE_URL ) ) {
+		if ( isset( $_POST['remote_data'] ) && filter_var( $_POST['remote_data'], FILTER_VALIDATE_URL ) ) {
 			$source = new Visualizer_Source_Csv_Remote( $_POST['remote_data'] );
 		} elseif ( isset( $_FILES['local_data'] ) && $_FILES['local_data']['error'] == 0 ) {
 			$source = new Visualizer_Source_Csv( $_FILES['local_data']['tmp_name'] );
@@ -488,7 +489,7 @@ public function uploadData() {
 		}
 
 		$render->render();
-		exit;
+		wp_die();
 	}
 
 	/**

index.php

@@ -90,32 +90,20 @@ function visualizer_launch() {
 		define( 'VISUALIZER_CSV_ENCLOSURE', '"' );
 	}
 
-	// don't load the plugin if cron job is running or doing autosave
-	$doing_autosave = defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE;
-	$doing_cron = defined( 'DOING_CRON' ) && DOING_CRON;
-	$doing_ajax = defined( 'DOING_AJAX' ) && DOING_AJAX;
-	if ( $doing_autosave || $doing_cron ) {
-		return;
-	}
-
 	// instantiate the plugin
 	$plugin = Visualizer_Plugin::instance();
 
 	// set general modules
 	$plugin->setModule( Visualizer_Module_Setup::NAME );
 	$plugin->setModule( Visualizer_Module_Sources::NAME );
 
-	if ( $doing_ajax ) {
-		// set ajax modules
-		$plugin->setModule( Visualizer_Module_Chart::NAME );
+	$plugin->setModule( Visualizer_Module_Chart::NAME );
+	if ( is_admin() ) {
+		// set admin modules
+		$plugin->setModule( Visualizer_Module_Admin::NAME );
 	} else {
-		if ( is_admin() ) {
-			// set admin modules
-			$plugin->setModule( Visualizer_Module_Admin::NAME );
-		} else {
-			// set frontend modules
-			$plugin->setModule( Visualizer_Module_Frontend::NAME );
-		}
+		// set frontend modules
+		$plugin->setModule( Visualizer_Module_Frontend::NAME );
 	}
 }
 

js/frame.js

@@ -31,7 +31,7 @@
                 parent.addClass('open');
             }
         });
-        $('#vz-import-file').click(function () {
+        $('#view-remote-file').click(function () {
             var url = $(this).parent().find('#remote-data').val();
 
             if (url !== '') {