Xcloner_Encryption::decrypt_file() - Code Metrics - Inspection of "scrutinizer fixes encryption" - ovidiul/XCloner-Wordpress - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — dev (6b2b1f)

by Liuta

created 2019-01-13 19:04 UTC

Xcloner_Encryption::decrypt_file() F

↳ Parent: Xcloner_Encryption

Complexity

Conditions	20
Paths	992

Size

Total Lines	116
Code Lines	60

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	3	Features	0

Metric	Value
cc	20
eloc	60
c	5
b	3
f	0
nc	992
nop	6
dl	0
loc	116
rs	0.0111

How to fix Long Method Complexity

<?php
/**
 * Created by PhpStorm.
 * User: thinkovi
 * Date: 2018-11-27
 * Time: 12:11
 */

//namespace XCloner;

class Xcloner_Encryption
{
	const FILE_ENCRYPTION_BLOCKS = 1024 * 1024;
	const FILE_ENCRYPTION_SUFFIX = ".encrypted";
	const FILE_DECRYPTION_SUFFIX = ".decrypted";

	private $xcloner_settings;
	private $logger;
	private $xcloner_container;
	private $verification = false;

	/**
	 * Xcloner_Encryption constructor.
	 * @param Xcloner $xcloner_container
	 */
	public function __construct(Xcloner $xcloner_container)
	{
		$this->xcloner_container = $xcloner_container;
		if (method_exists($xcloner_container, 'get_xcloner_settings')) {
			$this->xcloner_settings = $xcloner_container->get_xcloner_settings();
		} else {
			$this->xcloner_settings = "";
		}

		if (method_exists($xcloner_container, 'get_xcloner_logger')) {
			$this->logger = $xcloner_container->get_xcloner_logger()->withName("xcloner_encryption");
		} else {
			$this->logger = "";
		}
	}

	/**
	 * Returns the backup encryption key
	 *
	 * @return string|null
	 */
	public function get_backup_encryption_key()
	{
		if (is_object($this->xcloner_settings)) {
			return $this->xcloner_settings->get_xcloner_encryption_key();
		}

		return null;

	}

	/**
	 * Check if provided filename has encrypted suffix
	 *
	 * @param $filename
	 * @return bool
	 */
	public function is_encrypted_file($filename) {
		$fp = fopen($this->get_xcloner_path().$filename, 'r');
		if (is_resource($fp)) {
			$encryption_length = fread($fp, 16);
			fclose($fp);
			if (is_numeric($encryption_length)) {
				return true;
			}
		}

		return false;

	}

	/**
	 * Returns the filename with encrypted suffix
	 *
	 * @param string $filename
	 * @return string
	 */
	public function get_encrypted_target_backup_file_name($filename) {

		return str_replace(self::FILE_DECRYPTION_SUFFIX, "", $filename).self::FILE_ENCRYPTION_SUFFIX;
	}

	/**
	 * Returns the filename without encrypted suffix
	 *
	 * @param string $filename
	 * @return string
	 */
	public function get_decrypted_target_backup_file_name($filename) {

		return str_replace(self::FILE_ENCRYPTION_SUFFIX, "", $filename).self::FILE_DECRYPTION_SUFFIX;
	}

	/**
	 * Encrypt the passed file and saves the result in a new file with ".enc" as suffix.
	 *
	 * @param string $source Path to file that should be encrypted
	 * @param string $dest   File name where the encryped file should be written to.
	 * @param string $key    The key used for the encryption
	 * @param int $start   Start position for reading when doing incremental mode.
	 * @param string $iv   The IV key to use.
	 * @param bool $verification   Weather we should we try to verify the decryption.
	 * @return array|false  Returns array or FALSE if an error occured
     * @throws Exception
	 */
	public function encrypt_file($source, $dest = "", $key = "", $start = 0, $iv = 0, $verification = true, $recursive = false)
	{
		if (is_object($this->logger)) {
			$this->logger->info(sprintf('Encrypting file %s at position %d IV %s', $source, $start, base64_encode($iv)));
		}

		//$key = substr(sha1($key, true), 0, 16);
		if (!$key) {
			$key = $this->get_backup_encryption_key();
		}
		$key_digest = openssl_digest($key, "md5", true);

		$keep_local = 1;
		if (!$dest) {
			$dest = $this->get_encrypted_target_backup_file_name($source);
			$keep_local = 0;
		}

		if (!$iv || !$start) {
			//$iv = openssl_random_pseudo_bytes(16);
			$iv = str_pad(self::FILE_ENCRYPTION_BLOCKS, 16, "0000000000000000", STR_PAD_LEFT);
		}

		if (!$start) {
			$fpOut = fopen($this->get_xcloner_path().$dest, 'w');
		} else {
			$fpOut = fopen($this->get_xcloner_path().$dest, 'a');
		}

		if (is_resource($fpOut)) {

			// Put the initialization vector to the beginning of the file
			if (!$start) {
				fwrite($fpOut, $iv);
			}

			if (file_exists($this->get_xcloner_path().$source) &&
				$fpIn = fopen($this->get_xcloner_path().$source, 'rb')) {

				fseek($fpIn, (int)$start);

				if (!feof($fpIn)) {

					$plaintext = fread($fpIn, 16 * self::FILE_ENCRYPTION_BLOCKS);
					$ciphertext = openssl_encrypt($plaintext, 'AES-128-CBC', $key_digest, OPENSSL_RAW_DATA, $iv);

					// Use the first 16 bytes of the ciphertext as the next initialization vector
					$iv = substr($ciphertext, 0, 16);
					//$iv = openssl_random_pseudo_bytes(16);

					fwrite($fpOut, $ciphertext);

					$start = ftell($fpIn);

					fclose($fpOut);

					if (!feof($fpIn)) {
						fclose($fpIn);
						//echo "\n NEW:".$key.md5($iv);
						//self::encryptFile($source, $dest, $key, $start, $iv);
						if ($recursive) {
							$this->encrypt_file($source, $dest, $key, $start, ($iv), $verification, $recursive);
						} else {

							if (($iv) != base64_decode(base64_encode($iv)))
							{
								throw new \Exception('Could not encode IV for transport');
							}

							return array(
								"start" => $start,
								"iv" => base64_encode($iv),
								"target_file" => $dest,
								"finished" => 0
							);
						}
					}

				}
			} else {
				if (is_object($this->logger)) {
					$this->logger->error('Unable to read source file for encryption.');
				}
				throw new \Exception("Unable to read source file for encryption.");
			}
		} else {
			if (is_object($this->logger)) {
				$this->logger->error('Unable to write destination file for encryption.');
			}
			throw new \Exception("Unable to write destination file for encryption.");
		}

		if ($verification) {
			$this->verify_encrypted_file($dest);
		}

		//we replace the original backup with the encrypted one
		if (!$keep_local && copy($this->get_xcloner_path().$dest,
			 $this->get_xcloner_path().$source)) {
			unlink($this->get_xcloner_path().$dest);
		}


		return array("target_file" => $dest, "finished" => 1);
	}

	/**
	 * @param string $file
	 */
	public function verify_encrypted_file($file) {
		if (is_object($this->logger)) {
			$this->logger->info(sprintf('Verifying encrypted file %s', $file));
		}

		$this->verification = true;
		$this->decrypt_file($file);
		$this->verification = false;
	}

	/**
	 * Dencrypt the passed file and saves the result in a new file, removing the
	 * last 4 characters from file name.
	 *
	 * @param string $source Path to file that should be decrypted
	 * @param string $dest   File name where the decryped file should be written to.
	 * @param string $key    The key used for the decryption (must be the same as for encryption)
	 * @param int $start   Start position for reading when doing incremental mode.
	 * @param string $iv   The IV key to use.
	 * @return array|false  Returns array or FALSE if an error occured
     * @throws Exception
	 */
	public function decrypt_file($source, $dest = "", $key = "", $start = 0, $iv = 0, $recursive = false)
	{
		//if (is_object($this->logger)) {
			$this->logger->info(sprintf('Decrypting file %s at position %d with IV %s', $source, $start, base64_encode($iv)));
		//}

		//$key = substr(sha1($key, true), 0, 16);
		if (!$key) {
			$key = $this->get_backup_encryption_key();
		}

		$key_digest = openssl_digest($key, "md5", true);

		$keep_local = 1;
		if (!$dest) {
			$dest = $this->get_decrypted_target_backup_file_name($source);
			$keep_local = 0;
		}

		$write_mode = 'a';
		if (!$start) {
            $write_mode = 'w';
        }

        if ($this->verification) {
            $fpOut = fopen("php://stdout", $write_mode);
        } else {
            $fpOut = fopen($this->get_xcloner_path().$dest, $write_mode);
        }


		if (!$fpOut) {

            //if (is_object($this->logger)) {
                $this->logger->error('Unable to write destination file for decryption');
            //}
            throw new \Exception("Unable to write destination file for decryption");
        }


			if (file_exists($this->get_xcloner_path().$source) &&
				$fpIn = fopen($this->get_xcloner_path().$source, 'rb')) {

				$encryption_length = (int)fread($fpIn, 16);
				if (!$encryption_length) {
					$encryption_length = self::FILE_ENCRYPTION_BLOCKS;
				}

				fseek($fpIn, (int)$start);

				// Get the initialzation vector from the beginning of the file
				if (!$iv) {
					$iv = fread($fpIn, 16);
				}

				if (!feof($fpIn)) {

					// we have to read one block more for decrypting than for encrypting
					$ciphertext = fread($fpIn, 16 * ($encryption_length + 1));
					$plaintext = openssl_decrypt($ciphertext, 'AES-128-CBC', $key_digest, OPENSSL_RAW_DATA, $iv);

					if (!$plaintext) {
						unlink($this->get_xcloner_path().$dest);
						//if (is_object($this->logger)) {
							$this->logger->error('Backup decryption failed, please check your provided Encryption Key.');
						//}
						throw new \Exception("Backup decryption failed, please check your provided Encryption Key.");
					}

					// Use the first 16 bytes of the ciphertext as the next initialization vector
					$iv = substr($ciphertext, 0, 16);

					if (!$this->verification) {
						fwrite($fpOut, $plaintext);
					}

					$start = ftell($fpIn);

					fclose($fpOut);

					if (!feof($fpIn)) {
						fclose($fpIn);
						if ($this->verification || $recursive) {
							unset($ciphertext);
							unset($plaintext);
							$this->decrypt_file($source, $dest, $key, $start, $iv, $recursive);
						} else {
							if (($iv) != base64_decode(base64_encode($iv)))
							{
								throw new \Exception('Could not encode IV for transport');
							}

							return array(
								"start" => $start,
								"encryption_length" => $encryption_length,
								"iv" => base64_encode($iv),
								"target_file" => $dest,
								"finished" => 0
							);
					}
					}

				}
			} else {
				//if (is_object($this->logger)) {
					$this->logger->error('Unable to read source file for decryption');
				//}
				throw new \Exception("Unable to read source file for decryption");
			}

		//we replace the original backup with the encrypted one
		if (!$keep_local && !$this->verification && copy($this->get_xcloner_path().$dest,
			$this->get_xcloner_path().$source)) {
			unlink($this->get_xcloner_path().$dest);
		}

		return array("target_file" => $dest, "finished" => 1);
	}

	public function get_xcloner_path() {
		if (is_object($this->xcloner_settings)) {
			return $this->xcloner_settings->get_xcloner_store_path().DS;
		}

		return null;
	}

}


try {

	if (isset($argv[1])) {

		class Xcloner {
			/**
			 * Xcloner constructor.
			 */
			public function __construct()
			{
			}
		}
		$xcloner_encryption = new Xcloner_Encryption(new Xcloner());

		if ($argv[1] == "-e") {
			$xcloner_encryption->encrypt_file($argv[2], $argv[2].".enc", $argv[4], 0, 0, false, true);
		} elseif ($argv[1] == "-d") {
			$xcloner_encryption->decrypt_file($argv[2], $argv[2].".dec", $argv[4], 0, 0, true);
		}
	}
}catch (\Exception $e) {
	echo "CAUGHT: ".$e->getMessage();
}


1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: thinkovi
5			* Date: 2018-11-27
6			* Time: 12:11
7			*/
8
9			//namespace XCloner;
10
11			class Xcloner_Encryption
12			{
13			const FILE_ENCRYPTION_BLOCKS = 1024 * 1024;
14			const FILE_ENCRYPTION_SUFFIX = ".encrypted";
15			const FILE_DECRYPTION_SUFFIX = ".decrypted";
16
17			private $xcloner_settings;
18			private $logger;
19			private $xcloner_container;
20			private $verification = false;
21
22			/**
23			* Xcloner_Encryption constructor.
24			* @param Xcloner $xcloner_container
25			*/
26			public function __construct(Xcloner $xcloner_container)
27			{
28			$this->xcloner_container = $xcloner_container;
29			if (method_exists($xcloner_container, 'get_xcloner_settings')) {
30			$this->xcloner_settings = $xcloner_container->get_xcloner_settings();
31			} else {
32			$this->xcloner_settings = "";
33			}
34
35			if (method_exists($xcloner_container, 'get_xcloner_logger')) {
36			$this->logger = $xcloner_container->get_xcloner_logger()->withName("xcloner_encryption");
37			} else {
38			$this->logger = "";
39			}
40			}
41
42			/**
43			* Returns the backup encryption key
44			*
45			* @return string\|null
46			*/
47			public function get_backup_encryption_key()
48			{
49			if (is_object($this->xcloner_settings)) {
50			return $this->xcloner_settings->get_xcloner_encryption_key();
51			}
52
53			return null;
54
55			}
56
57			/**
58			* Check if provided filename has encrypted suffix
59			*
60			* @param $filename
61			* @return bool
62			*/
63			public function is_encrypted_file($filename) {
64			$fp = fopen($this->get_xcloner_path().$filename, 'r');
65			if (is_resource($fp)) {
66			$encryption_length = fread($fp, 16);
67			fclose($fp);
68			if (is_numeric($encryption_length)) {
69			return true;
70			}
71			}
72
73			return false;
74
75			}
76
77			/**
78			* Returns the filename with encrypted suffix
79			*
80			* @param string $filename
81			* @return string
82			*/
83			public function get_encrypted_target_backup_file_name($filename) {
84
85			return str_replace(self::FILE_DECRYPTION_SUFFIX, "", $filename).self::FILE_ENCRYPTION_SUFFIX;
86			}
87
88			/**
89			* Returns the filename without encrypted suffix
90			*
91			* @param string $filename
92			* @return string
93			*/
94			public function get_decrypted_target_backup_file_name($filename) {
95
96			return str_replace(self::FILE_ENCRYPTION_SUFFIX, "", $filename).self::FILE_DECRYPTION_SUFFIX;
97			}
98
99			/**
100			* Encrypt the passed file and saves the result in a new file with ".enc" as suffix.
101			*
102			* @param string $source Path to file that should be encrypted
103			* @param string $dest File name where the encryped file should be written to.
104			* @param string $key The key used for the encryption
105			* @param int $start Start position for reading when doing incremental mode.
106			* @param string $iv The IV key to use.
107			* @param bool $verification Weather we should we try to verify the decryption.
108			* @return array\|false Returns array or FALSE if an error occured
109			* @throws Exception
110			*/
111			public function encrypt_file($source, $dest = "", $key = "", $start = 0, $iv = 0, $verification = true, $recursive = false)
112			{
113			if (is_object($this->logger)) {
114			$this->logger->info(sprintf('Encrypting file %s at position %d IV %s', $source, $start, base64_encode($iv)));
115			}
116
117			//$key = substr(sha1($key, true), 0, 16);
118			if (!$key) {
119			$key = $this->get_backup_encryption_key();
120			}
121			$key_digest = openssl_digest($key, "md5", true);
122
123			$keep_local = 1;
124			if (!$dest) {
125			$dest = $this->get_encrypted_target_backup_file_name($source);
126			$keep_local = 0;
127			}
128
129			if (!$iv \|\| !$start) {
130			//$iv = openssl_random_pseudo_bytes(16);
131			$iv = str_pad(self::FILE_ENCRYPTION_BLOCKS, 16, "0000000000000000", STR_PAD_LEFT);
132			}
133
134			if (!$start) {
135			$fpOut = fopen($this->get_xcloner_path().$dest, 'w');
136			} else {
137			$fpOut = fopen($this->get_xcloner_path().$dest, 'a');
138			}
139
140			if (is_resource($fpOut)) {
141
142			// Put the initialization vector to the beginning of the file
143			if (!$start) {
144			fwrite($fpOut, $iv);
145			}
146
147			if (file_exists($this->get_xcloner_path().$source) &&
148			$fpIn = fopen($this->get_xcloner_path().$source, 'rb')) {
149
150			fseek($fpIn, (int)$start);
151
152			if (!feof($fpIn)) {
153
154			$plaintext = fread($fpIn, 16 * self::FILE_ENCRYPTION_BLOCKS);
155			$ciphertext = openssl_encrypt($plaintext, 'AES-128-CBC', $key_digest, OPENSSL_RAW_DATA, $iv);
156
157			// Use the first 16 bytes of the ciphertext as the next initialization vector
158			$iv = substr($ciphertext, 0, 16);
159			//$iv = openssl_random_pseudo_bytes(16);
160
161			fwrite($fpOut, $ciphertext);
162
163			$start = ftell($fpIn);
164
165			fclose($fpOut);
166
167			if (!feof($fpIn)) {
168			fclose($fpIn);
169			//echo "\n NEW:".$key.md5($iv);
170			//self::encryptFile($source, $dest, $key, $start, $iv);
171			if ($recursive) {
172			$this->encrypt_file($source, $dest, $key, $start, ($iv), $verification, $recursive);
173			} else {
174
175			if (($iv) != base64_decode(base64_encode($iv)))
176			{
177			throw new \Exception('Could not encode IV for transport');
178			}
179
180			return array(
181			"start" => $start,
182			"iv" => base64_encode($iv),
183			"target_file" => $dest,
184			"finished" => 0
185			);
186			}
187			}
188
189			}
190			} else {
191			if (is_object($this->logger)) {
192			$this->logger->error('Unable to read source file for encryption.');
193			}
194			throw new \Exception("Unable to read source file for encryption.");
195			}
196			} else {
197			if (is_object($this->logger)) {
198			$this->logger->error('Unable to write destination file for encryption.');
199			}
200			throw new \Exception("Unable to write destination file for encryption.");
201			}
202
203			if ($verification) {
204			$this->verify_encrypted_file($dest);
205			}
206
207			//we replace the original backup with the encrypted one
208			if (!$keep_local && copy($this->get_xcloner_path().$dest,
209			$this->get_xcloner_path().$source)) {
210			unlink($this->get_xcloner_path().$dest);
211			}
212
213
214			return array("target_file" => $dest, "finished" => 1);
215			}
216
217			/**
218			* @param string $file
219			*/
220			public function verify_encrypted_file($file) {
221			if (is_object($this->logger)) {
222			$this->logger->info(sprintf('Verifying encrypted file %s', $file));
223			}
224
225			$this->verification = true;
226			$this->decrypt_file($file);
227			$this->verification = false;
228			}
229
230			/**
231			* Dencrypt the passed file and saves the result in a new file, removing the
232			* last 4 characters from file name.
233			*
234			* @param string $source Path to file that should be decrypted
235			* @param string $dest File name where the decryped file should be written to.
236			* @param string $key The key used for the decryption (must be the same as for encryption)
237			* @param int $start Start position for reading when doing incremental mode.
238			* @param string $iv The IV key to use.
239			* @return array\|false Returns array or FALSE if an error occured
240			* @throws Exception
241			*/
242			public function decrypt_file($source, $dest = "", $key = "", $start = 0, $iv = 0, $recursive = false)
243			{
244			//if (is_object($this->logger)) {
245			$this->logger->info(sprintf('Decrypting file %s at position %d with IV %s', $source, $start, base64_encode($iv)));
246			//}
247
248			//$key = substr(sha1($key, true), 0, 16);
249			if (!$key) {
250			$key = $this->get_backup_encryption_key();
251			}
252
253			$key_digest = openssl_digest($key, "md5", true);
254
255			$keep_local = 1;
256			if (!$dest) {
257			$dest = $this->get_decrypted_target_backup_file_name($source);
258			$keep_local = 0;
259			}
260
261			$write_mode = 'a';
262			if (!$start) {
263			$write_mode = 'w';
264			}
265
266			if ($this->verification) {
267			$fpOut = fopen("php://stdout", $write_mode);
268			} else {
269			$fpOut = fopen($this->get_xcloner_path().$dest, $write_mode);
270			}
271
272
273			if (!$fpOut) {
			0 ignored issues – show introduced 2019-01-13 19:07 UTC by Report Bug Copy Issue Report `$fpOut` is of type `false\|resource`, thus it always evaluated to `false`. Loading history...
274			//if (is_object($this->logger)) {
275			$this->logger->error('Unable to write destination file for decryption');
276			//}
277			throw new \Exception("Unable to write destination file for decryption");
278			}
279
280
281			if (file_exists($this->get_xcloner_path().$source) &&
282			$fpIn = fopen($this->get_xcloner_path().$source, 'rb')) {
283
284			$encryption_length = (int)fread($fpIn, 16);
285			if (!$encryption_length) {
286			$encryption_length = self::FILE_ENCRYPTION_BLOCKS;
287			}
288
289			fseek($fpIn, (int)$start);
290
291			// Get the initialzation vector from the beginning of the file
292			if (!$iv) {
293			$iv = fread($fpIn, 16);
294			}
295
296			if (!feof($fpIn)) {
297
298			// we have to read one block more for decrypting than for encrypting
299			$ciphertext = fread($fpIn, 16 * ($encryption_length + 1));
300			$plaintext = openssl_decrypt($ciphertext, 'AES-128-CBC', $key_digest, OPENSSL_RAW_DATA, $iv);
301
302			if (!$plaintext) {
303			unlink($this->get_xcloner_path().$dest);
304			//if (is_object($this->logger)) {
305			$this->logger->error('Backup decryption failed, please check your provided Encryption Key.');
306			//}
307			throw new \Exception("Backup decryption failed, please check your provided Encryption Key.");
308			}
309
310			// Use the first 16 bytes of the ciphertext as the next initialization vector
311			$iv = substr($ciphertext, 0, 16);
312
313			if (!$this->verification) {
314			fwrite($fpOut, $plaintext);
315			}
316
317			$start = ftell($fpIn);
318
319			fclose($fpOut);
320
321			if (!feof($fpIn)) {
322			fclose($fpIn);
323			if ($this->verification \|\| $recursive) {
324			unset($ciphertext);
325			unset($plaintext);
326			$this->decrypt_file($source, $dest, $key, $start, $iv, $recursive);
327			} else {
328			if (($iv) != base64_decode(base64_encode($iv)))
329			{
330			throw new \Exception('Could not encode IV for transport');
331			}
332
333			return array(
334			"start" => $start,
335			"encryption_length" => $encryption_length,
336			"iv" => base64_encode($iv),
337			"target_file" => $dest,
338			"finished" => 0
339			);
340			}
341			}
342
343			}
344			} else {
345			//if (is_object($this->logger)) {
346			$this->logger->error('Unable to read source file for decryption');
347			//}
348			throw new \Exception("Unable to read source file for decryption");
349			}
350
351			//we replace the original backup with the encrypted one
352			if (!$keep_local && !$this->verification && copy($this->get_xcloner_path().$dest,
353			$this->get_xcloner_path().$source)) {
354			unlink($this->get_xcloner_path().$dest);
355			}
356
357			return array("target_file" => $dest, "finished" => 1);
358			}
359
360			public function get_xcloner_path() {
361			if (is_object($this->xcloner_settings)) {
362			return $this->xcloner_settings->get_xcloner_store_path().DS;
363			}
364
365			return null;
366			}
367
368			}
369
370
371			try {
372
373			if (isset($argv[1])) {
374
375			class Xcloner {
376			/**
377			* Xcloner constructor.
378			*/
379			public function __construct()
380			{
381			}
382			}
383			$xcloner_encryption = new Xcloner_Encryption(new Xcloner());
384
385			if ($argv[1] == "-e") {
386			$xcloner_encryption->encrypt_file($argv[2], $argv[2].".enc", $argv[4], 0, 0, false, true);
387			} elseif ($argv[1] == "-d") {
388			$xcloner_encryption->decrypt_file($argv[2], $argv[2].".dec", $argv[4], 0, 0, true);
389			}
390			}
391			}catch (\Exception $e) {
392			echo "CAUGHT: ".$e->getMessage();
393			}
394

ovidiul / XCloner-Wordpress

GitHub Access Token became invalid

Branch — dev (6b2b1f)

Xcloner_Encryption::decrypt_file() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like