ExcelExport::clearName() - Code Metrics - amarcinkowski/hospitalplugin - Measure and Improve Code Quality continuously with Scrutinizer

ExcelExport::clearName() A
last analyzed 2016-03-15 08:59 UTC

↳ Parent: ExcelExport

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	9
rs	9.6666
cc	2
eloc	7
nc	2
nop	2

<?php

namespace Hospitalplugin\utils;

use Hospitalplugin\Entities\InfectionsCRUD;
use Hospitalplugin\Entities\Infections;

class ExcelExport {
	
	/**
	 */
	public static function init() {
		add_action ( 'admin_init', array (
				'Hospitalplugin\utils\ExcelExport',
				'excel_export' 
		), 1 );
	}
	/**
	 * 
	 */
	public static function getColumnLetter($num) {
		return chr ( 65 + $num );
	}
	/**
	 */
	static function excel_export() {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
		if (! isset ( $_POST ['excel'] )) {
			return;
		} else {
			// dataSetName: Infections, Punction
			$dataSetName = $_POST ['dataSetName'];
			$classname = '\Hospitalplugin\utils\ExcelExport' . $dataSetName;
		}
		
		$objPHPExcel = new \PHPExcel ();
		$classname::fillData ( $objPHPExcel );
		
		ExcelExport::downloadExcel ( $objPHPExcel, $dataSetName . '.xlsx' );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}
	/**
	 */
	static function cellColor($objPHPExcel, $cells, $color) {

		$objPHPExcel->getActiveSheet ()->getStyle ( $cells )->getFill ()->applyFromArray ( array (
				'type' => \PHPExcel_Style_Fill::FILL_SOLID,
				'startcolor' => array (
						'rgb' => $color 
				) 
		) );
	}
	/**
	 */
	static function styleActiveSheet($objPHPExcel) {

		$objPHPExcel->getActiveSheet ()->getDefaultStyle ()->getFont ()->setName ( 'Arial' )->setSize ( 8 )->setBold ( false );
		$objPHPExcel->getActiveSheet ()->getDefaultStyle ()->getNumberFormat ()->setFormatCode ( \PHPExcel_Style_NumberFormat::FORMAT_TEXT );
		$objPHPExcel->getActiveSheet ()->freezePane ( 'A3' );
		$style = array (
				'alignment' => array (
						'horizontal' => \PHPExcel_Style_Alignment::HORIZONTAL_CENTER,
						'vertical' => \PHPExcel_Style_Alignment::VERTICAL_CENTER 
				) 
		);
		$objWorksheet = $objPHPExcel->getActiveSheet ();
		$objWorksheet->getDefaultStyle ()->applyFromArray ( $style );
		$objWorksheet->getStyle ( 'A1:AA2' )->getFont ()->setBold ( true );
		for($col = ord ( 'a' ); $col <= ord ( 'z' ); $col ++) {
			$objWorksheet->getColumnDimension ( chr ( $col ) )->setAutoSize ( true );
		}
		return $objPHPExcel;
	}
	/**
	 *
	 * @param unknown $objPHPExcel        	
	 */
	private static function downloadExcel($objPHPExcel, $filename) {
		ob_end_clean ();
		ob_start ();
		header ( "Content-type: application/vnd.ms-excel; charset=utf-8" );
		header ( "Content-Transfer-Encoding: binary" );
		header ( "Content-Description: File Transfer" );
		header ( "Content-Disposition: attachment; filename=\"" . $filename . "\"" );
		header ( "Cache-Control: max-age=0" );
		header ( "Expires: 0" );
		header ( "Pragma: no-cache" );
		
		$objWriter = \PHPExcel_IOFactory::createWriter ( $objPHPExcel, 'Excel2007' );
		$objWriter->setPreCalculateFormulas ( true );
		$objWriter->save ( 'php://output' );
		exit ();

	}
	/**
	 * remove special chars and words from string 
	 * @param unknown $string        	
	 */
	private static function clearName($string, $removeWords) {
		$string = str_replace ( ' ', '-', $string );
		$string = preg_replace ( '/[^A-Za-z0-9\-]/', '', $string );
		foreach ( $removeWords as $word ) {
			$string = str_replace ( $word, '', $string );
		}
		$string = substr ( $string, 0, 29 );
		return $string;
	}
	/**
	 * @param unknown $objPHPExcel        	
	 * @param unknown $title        	
	 */
	static function printTitle($objPHPExcel, $title) {

		$objPHPExcel->getActiveSheet ()->setTitle ( ExcelExport::clearName ( $title, array('ddzia') ), true );
		$objPHPExcel->getActiveSheet ()->setCellValueByColumnAndRow ( 0, 1, $title );
	}
	/**
	 *
	 * @param unknown $objPHPExcel
	 */
	static function newSheet($objPHPExcel, $index = -1) {

		if ($index < 0) {
			$index = $objPHPExcel->getSheetCount ();
		}
		$objPHPExcel->createSheet ( $index );
		$objPHPExcel->setActiveSheetIndex ( $index );
		$sheet = $objPHPExcel->getActiveSheet ();
		return $sheet;
	}
	
}

1			<?php
2
3			namespace Hospitalplugin\utils;
4
5			use Hospitalplugin\Entities\InfectionsCRUD;
6			use Hospitalplugin\Entities\Infections;
7
8			class ExcelExport {
9
10			/**
11			*/
12			public static function init() {
13			add_action ( 'admin_init', array (
14			'Hospitalplugin\utils\ExcelExport',
15			'excel_export'
16			), 1 );
17			}
18			/**
19			*
20			*/
21			public static function getColumnLetter($num) {
22			return chr ( 65 + $num );
23			}
24			/**
25			*/
26			static function excel_export() {
			0 ignored issues – show Best Practice introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Coding Style introduced 2015-11-23 17:01 UTC by Report Bug Copy Issue Report `excel_export` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
27			if (! isset ( $_POST ['excel'] )) {
28			return;
29			} else {
30			// dataSetName: Infections, Punction
31			$dataSetName = $_POST ['dataSetName'];
32			$classname = '\Hospitalplugin\utils\ExcelExport' . $dataSetName;
33			}
34
35			$objPHPExcel = new \PHPExcel ();
36			$classname::fillData ( $objPHPExcel );
37
38			ExcelExport::downloadExcel ( $objPHPExcel, $dataSetName . '.xlsx' );
			0 ignored issues – show Documentation introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report `$objPHPExcel` is of type `object<PHPExcel>`, but the function expects a `object<Hospitalplugin\utils\unknown>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
39			}
40			/**
41			*/
42			static function cellColor($objPHPExcel, $cells, $color) {
			0 ignored issues – show Best Practice introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
43			$objPHPExcel->getActiveSheet ()->getStyle ( $cells )->getFill ()->applyFromArray ( array (
44			'type' => \PHPExcel_Style_Fill::FILL_SOLID,
45			'startcolor' => array (
46			'rgb' => $color
47			)
48			) );
49			}
50			/**
51			*/
52			static function styleActiveSheet($objPHPExcel) {
			0 ignored issues – show Best Practice introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
53			$objPHPExcel->getActiveSheet ()->getDefaultStyle ()->getFont ()->setName ( 'Arial' )->setSize ( 8 )->setBold ( false );
54			$objPHPExcel->getActiveSheet ()->getDefaultStyle ()->getNumberFormat ()->setFormatCode ( \PHPExcel_Style_NumberFormat::FORMAT_TEXT );
55			$objPHPExcel->getActiveSheet ()->freezePane ( 'A3' );
56			$style = array (
57			'alignment' => array (
58			'horizontal' => \PHPExcel_Style_Alignment::HORIZONTAL_CENTER,
59			'vertical' => \PHPExcel_Style_Alignment::VERTICAL_CENTER
60			)
61			);
62			$objWorksheet = $objPHPExcel->getActiveSheet ();
63			$objWorksheet->getDefaultStyle ()->applyFromArray ( $style );
64			$objWorksheet->getStyle ( 'A1:AA2' )->getFont ()->setBold ( true );
65			for($col = ord ( 'a' ); $col <= ord ( 'z' ); $col ++) {
66			$objWorksheet->getColumnDimension ( chr ( $col ) )->setAutoSize ( true );
67			}
68			return $objPHPExcel;
69			}
70			/**
71			*
72			* @param unknown $objPHPExcel
73			*/
74			private static function downloadExcel($objPHPExcel, $filename) {
75			ob_end_clean ();
76			ob_start ();
77			header ( "Content-type: application/vnd.ms-excel; charset=utf-8" );
78			header ( "Content-Transfer-Encoding: binary" );
79			header ( "Content-Description: File Transfer" );
80			header ( "Content-Disposition: attachment; filename=\"" . $filename . "\"" );
81			header ( "Cache-Control: max-age=0" );
82			header ( "Expires: 0" );
83			header ( "Pragma: no-cache" );
84
85			$objWriter = \PHPExcel_IOFactory::createWriter ( $objPHPExcel, 'Excel2007' );
86			$objWriter->setPreCalculateFormulas ( true );
87			$objWriter->save ( 'php://output' );
88			exit ();
			0 ignored issues – show Coding Style Compatibility introduced 2015-11-23 17:01 UTC by Report Bug Copy Issue Report The method `downloadExcel()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
89			}
90			/**
91			* remove special chars and words from string
92			* @param unknown $string
93			*/
94			private static function clearName($string, $removeWords) {
95			$string = str_replace ( ' ', '-', $string );
96			$string = preg_replace ( '/[^A-Za-z0-9\-]/', '', $string );
97			foreach ( $removeWords as $word ) {
98			$string = str_replace ( $word, '', $string );
99			}
100			$string = substr ( $string, 0, 29 );
101			return $string;
102			}
103			/**
104			* @param unknown $objPHPExcel
105			* @param unknown $title
106			*/
107			static function printTitle($objPHPExcel, $title) {
			0 ignored issues – show Best Practice introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
108			$objPHPExcel->getActiveSheet ()->setTitle ( ExcelExport::clearName ( $title, array('ddzia') ), true );
109			$objPHPExcel->getActiveSheet ()->setCellValueByColumnAndRow ( 0, 1, $title );
110			}
111			/**
112			*
113			* @param unknown $objPHPExcel
114			*/
115			static function newSheet($objPHPExcel, $index = -1) {
			0 ignored issues – show Best Practice introduced 2015-11-23 17:00 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
116			if ($index < 0) {
117			$index = $objPHPExcel->getSheetCount ();
118			}
119			$objPHPExcel->createSheet ( $index );
120			$objPHPExcel->setActiveSheetIndex ( $index );
121			$sheet = $objPHPExcel->getActiveSheet ();
122			return $sheet;
123			}
124
125			}

amarcinkowski / hospitalplugin

ExcelExport::clearName() A last analyzed 2016-03-15 08:59 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ExcelExport::clearName() A
last analyzed 2016-03-15 08:59 UTC