Dolibarr / dolibarr

test/phpunit/CommonClassTest.class.php

Spacing +17 added lines, -17 removed lines

@@ -31,7 +31,7 @@ 
 	$_SERVER['PHP_SELF'] = "phpunit";
 }
 
-global $conf,$user,$langs,$db;
+global $conf, $user, $langs, $db;
 //define('TEST_DB_FORCE_TYPE','mysql');	// This is to force using mysql driver
 //require_once 'PHPUnit/Autoload.php';
 require_once dirname(__FILE__).'/../../htdocs/master.inc.php';
@@ -91,7 +91,7 @@ 
 		parent::__construct($name, $data, $dataName);
 
 		//$this->sharedFixture
-		global $conf,$user,$langs,$db;
+		global $conf, $user, $langs, $db;
 		$this->savconf = $conf;
 		$this->savuser = $user;
 		$this->savlangs = $langs;
@@ -110,7 +110,7 @@ 
 	 */
 	public static function setUpBeforeClass(): void
 	{
-		global $conf,$user,$langs,$db;
+		global $conf, $user, $langs, $db;
 		$db->begin(); // This is to have all actions inside a transaction even if test launched without suite.
 
 		if ((int) getenv('PHPUNIT_DEBUG') > 0) {
@@ -193,7 +193,7 @@ 
 	 */
 	protected function setUp(): void
 	{
-		global $conf,$user,$langs,$db;
+		global $conf, $user, $langs, $db;
 
 		$conf = $this->savconf;
 		$user = $this->savuser;
@@ -287,7 +287,7 @@ 
 				if (in_array($sKey, $fieldstoignorearray)) {
 					continue;
 				}
-				if (! $ignoretype && ($oVarsA[$sKey] !== $oVarsB[$sKey])) {
+				if (!$ignoretype && ($oVarsA[$sKey] !== $oVarsB[$sKey])) {
 					$retAr[] = get_class($oA).'::'.$sKey.' : '.(is_object($oVarsA[$sKey]) ? get_class($oVarsA[$sKey]) : json_encode($oVarsA[$sKey])).' <> '.(is_object($oVarsB[$sKey]) ? get_class($oVarsB[$sKey]) : json_encode($oVarsB[$sKey]));
 				}
 				if ($ignoretype && ($oVarsA[$sKey] != $oVarsB[$sKey])) {
@@ -339,7 +339,7 @@ 
 		'accounting' => 'Accounting',
 		'agenda' => 'Agenda',
 		'ai' => 'Ai',
-		'anothermodule' => null,  // Not used in code, used in translations.lang
+		'anothermodule' => null, // Not used in code, used in translations.lang
 		'api' => 'Api',
 		'asset' => 'Asset',
 		'bank' => 'Banque',
@@ -351,9 +351,9 @@ 
 		'cashdesk' => null,
 		'category' => 'Categorie',
 		'clicktodial' => 'ClickToDial',
-		'collab' => 'Collab',  // TODO: fill in proper name
+		'collab' => 'Collab', // TODO: fill in proper name
 		'comptabilite' => 'Comptabilite',
-		'contact' => null,  // TODO: fill in proper class
+		'contact' => null, // TODO: fill in proper class
 		'contract' => 'Contrat',
 		'cron' => 'Cron',
 		'datapolicy' => 'DataPolicy',
@@ -361,22 +361,22 @@ 
 		'debugbar' => 'DebugBar',
 		'shipping' => 'Expedition',
 		'deplacement' => 'Deplacement',
-		"documentgeneration" => 'DocumentGeneration',  // TODO: fill in proper name
+		"documentgeneration" => 'DocumentGeneration', // TODO: fill in proper name
 		'don' => 'Don',
 		'dynamicprices' => 'DynamicPrices',
 		'ecm' => 'ECM',
-		'ecotax' => null,  // TODO: External module ?
+		'ecotax' => null, // TODO: External module ?
 		'emailcollector' => 'EmailCollector',
 		'eventorganization' => 'EventOrganization',
 		'expensereport' => 'ExpenseReport',
 		'export' => 'Export',
-		'externalrss' => 'ExternalRss',  // TODO: fill in proper name
+		'externalrss' => 'ExternalRss', // TODO: fill in proper name
 		'externalsite' => 'ExternalSite',
 		'fckeditor' => 'Fckeditor',
 		'fournisseur' => 'Fournisseur',
 		'ftp' => 'FTP',
-		'geoipmaxmind' => 'GeoIPMaxmind',  // TODO: fill in proper name
-		'google' => null,  // External ?
+		'geoipmaxmind' => 'GeoIPMaxmind', // TODO: fill in proper name
+		'google' => null, // External ?
 		'gravatar' => 'Gravatar',
 		'holiday' => 'Holiday',
 		'hrm' => 'HRM',
@@ -390,7 +390,7 @@ 
 		'ldap' => 'Ldap',
 		'loan' => 'Loan',
 		'mailing' => 'Mailing',
-		'mailman' => null,  // Same module as mailmanspip -> MailmanSpip ??
+		'mailman' => null, // Same module as mailmanspip -> MailmanSpip ??
 		'mailmanspip' => 'MailmanSpip',
 		'margin' => 'Margin',
 		'member' => 'Adherent',
@@ -403,7 +403,7 @@ 
 		'notification' => 'Notification',
 		'numberwords' => null, // Not provided by default, no module tests
 		'oauth' => 'Oauth',
-		'openstreetmap' => null,  // External module?
+		'openstreetmap' => null, // External module?
 		'opensurvey' => 'OpenSurvey',
 		'order' => 'Commande',
 		'partnership' => 'Partnership',
@@ -430,8 +430,8 @@ 
 		'stock' => 'Stock',
 		'stocktransfer' => 'StockTransfer',
 		'stripe' => 'Stripe',
-		'supplier_invoice' => null,  // Special case, uses invoice
-		'supplier_order' => null,  // Special case, uses invoice
+		'supplier_invoice' => null, // Special case, uses invoice
+		'supplier_order' => null, // Special case, uses invoice
 		'supplier_proposal' => 'SupplierProposal',
 		'syslog' => 'Syslog',
 		'takepos' => 'TakePos',