Completed
Push — master ( bbafe0...451020 )
by Aimeos
06:59
created
controller/jobs/src/Controller/Jobs/Order/Service/Payment/Factory.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string|null $name Name of the controller or "Standard" if null
31 31
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
32 32
 	 */
33
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
33
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
34 34
 	{
35 35
 		/** controller/jobs/order/service/payment/name
36 36
 		 * Class name of the used order service payment scheduler controller implementation
@@ -65,20 +65,20 @@  discard block
 block discarded – undo
65 65
 		 * @since 2014.07
66 66
 		 * @category Developer
67 67
 		 */
68
-		if( $name === null ) {
69
-			$name = $context->getConfig()->get( 'controller/jobs/order/service/payment/name', 'Standard' );
68
+		if ($name === null) {
69
+			$name = $context->getConfig()->get('controller/jobs/order/service/payment/name', 'Standard');
70 70
 		}
71 71
 
72
-		if( ctype_alnum( $name ) === false )
72
+		if (ctype_alnum($name) === false)
73 73
 		{
74
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Payment\\' . $name : '<not a string>';
75
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
74
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Payment\\' . $name : '<not a string>';
75
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
76 76
 		}
77 77
 
78 78
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
79 79
 		$classname = '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Payment\\' . $name;
80 80
 
81
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
81
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
82 82
 
83 83
 		/** controller/jobs/order/service/payment/decorators/excludes
84 84
 		 * Excludes decorators added by the "common" option from the order service payment controllers
@@ -154,6 +154,6 @@  discard block
 block discarded – undo
154 154
 		 * @see controller/jobs/order/service/payment/decorators/excludes
155 155
 		 * @see controller/jobs/order/service/payment/decorators/global
156 156
 		 */
157
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'order/service/payment' );
157
+		return self::addControllerDecorators($context, $aimeos, $controller, 'order/service/payment');
158 158
 	}
159 159
 }
160 160
\ No newline at end of file
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Order/Service/Payment/Standard.php 2 patches
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	public function getName()
31 31
 	{
32
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Capture authorized payments' );
32
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Capture authorized payments');
33 33
 	}
34 34
 
35 35
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getDescription()
42 42
 	{
43
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Authorized payments of orders will be captured after dispatching or after a configurable amount of time' );
43
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Authorized payments of orders will be captured after dispatching or after a configurable amount of time');
44 44
 	}
45 45
 
46 46
 
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
 		 * @category User
69 69
 		 * @category Developer
70 70
 		 */
71
-		$days = $config->get( 'controller/jobs/order/service/payment/limit-days', 90 );
72
-		$date = date( 'Y-m-d 00:00:00', time() - 86400 * $days );
71
+		$days = $config->get('controller/jobs/order/service/payment/limit-days', 90);
72
+		$date = date('Y-m-d 00:00:00', time() - 86400 * $days);
73 73
 
74 74
 		/** controller/jobs/order/service/payment/capture-days
75 75
 		 * Automatically capture payments after the configured amount of days
@@ -84,91 +84,91 @@  discard block
 block discarded – undo
84 84
 		 * @category User
85 85
 		 * @category Developer
86 86
 		 */
87
-		$capDays = $config->get( 'controller/jobs/order/service/payment/capture-days', null );
87
+		$capDays = $config->get('controller/jobs/order/service/payment/capture-days', null);
88 88
 
89 89
 
90
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
90
+		$serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service');
91 91
 		$serviceSearch = $serviceManager->createSearch();
92
-		$serviceSearch->setConditions( $serviceSearch->compare( '==', 'service.type.code', 'payment' ) );
92
+		$serviceSearch->setConditions($serviceSearch->compare('==', 'service.type.code', 'payment'));
93 93
 
94
-		$orderManager = \Aimeos\MShop\Factory::createManager( $context, 'order' );
94
+		$orderManager = \Aimeos\MShop\Factory::createManager($context, 'order');
95 95
 		$orderSearch = $orderManager->createSearch();
96 96
 
97
-		$status = array( \Aimeos\MShop\Order\Item\Base::STAT_DISPATCHED, \Aimeos\MShop\Order\Item\Base::STAT_DELIVERED );
97
+		$status = array(\Aimeos\MShop\Order\Item\Base::STAT_DISPATCHED, \Aimeos\MShop\Order\Item\Base::STAT_DELIVERED);
98 98
 		$start = 0;
99 99
 
100 100
 		do
101 101
 		{
102
-			$serviceItems = $serviceManager->searchItems( $serviceSearch );
102
+			$serviceItems = $serviceManager->searchItems($serviceSearch);
103 103
 
104
-			foreach( $serviceItems as $serviceItem )
104
+			foreach ($serviceItems as $serviceItem)
105 105
 			{
106 106
 				try
107 107
 				{
108
-					$serviceProvider = $serviceManager->getProvider( $serviceItem );
108
+					$serviceProvider = $serviceManager->getProvider($serviceItem);
109 109
 
110
-					if( !$serviceProvider->isImplemented( \Aimeos\MShop\Service\Provider\Payment\Base::FEAT_CAPTURE ) ) {
110
+					if (!$serviceProvider->isImplemented(\Aimeos\MShop\Service\Provider\Payment\Base::FEAT_CAPTURE)) {
111 111
 						continue;
112 112
 					}
113 113
 
114 114
 
115 115
 					$expr = array();
116
-					$expr[] = $orderSearch->compare( '==', 'order.siteid', $serviceItem->getSiteId() );
117
-					$expr[] = $orderSearch->compare( '>', 'order.datepayment', $date );
116
+					$expr[] = $orderSearch->compare('==', 'order.siteid', $serviceItem->getSiteId());
117
+					$expr[] = $orderSearch->compare('>', 'order.datepayment', $date);
118 118
 
119
-					if( $capDays !== null )
119
+					if ($capDays !== null)
120 120
 					{
121
-						$capdate = date( 'Y-m-d 00:00:00', time() - 86400 * $capDays );
122
-						$expr[] = $orderSearch->compare( '<=', 'order.datepayment', $capdate );
121
+						$capdate = date('Y-m-d 00:00:00', time() - 86400 * $capDays);
122
+						$expr[] = $orderSearch->compare('<=', 'order.datepayment', $capdate);
123 123
 					}
124 124
 					else
125 125
 					{
126
-						$expr[] = $orderSearch->compare( '==', 'order.statusdelivery', $status );
126
+						$expr[] = $orderSearch->compare('==', 'order.statusdelivery', $status);
127 127
 					}
128 128
 
129
-					$expr[] = $orderSearch->compare( '==', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_AUTHORIZED );
130
-					$expr[] = $orderSearch->compare( '==', 'order.base.service.code', $serviceItem->getCode() );
131
-					$expr[] = $orderSearch->compare( '==', 'order.base.service.type', 'payment' );
129
+					$expr[] = $orderSearch->compare('==', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_AUTHORIZED);
130
+					$expr[] = $orderSearch->compare('==', 'order.base.service.code', $serviceItem->getCode());
131
+					$expr[] = $orderSearch->compare('==', 'order.base.service.type', 'payment');
132 132
 
133
-					$orderSearch->setConditions( $orderSearch->combine( '&&', $expr ) );
133
+					$orderSearch->setConditions($orderSearch->combine('&&', $expr));
134 134
 
135 135
 
136 136
 					$orderStart = 0;
137 137
 
138 138
 					do
139 139
 					{
140
-						$orderItems = $orderManager->searchItems( $orderSearch );
140
+						$orderItems = $orderManager->searchItems($orderSearch);
141 141
 
142
-						foreach( $orderItems as $orderItem )
142
+						foreach ($orderItems as $orderItem)
143 143
 						{
144 144
 							try
145 145
 							{
146
-								$serviceProvider->capture( $orderItem );
146
+								$serviceProvider->capture($orderItem);
147 147
 							}
148
-							catch( \Exception $e )
148
+							catch (\Exception $e)
149 149
 							{
150 150
 								$str = 'Error while capturing payment for order with ID "%1$s": %2$s';
151
-								$context->getLogger()->log( sprintf( $str, $orderItem->getId(), $e->getMessage() ) );
151
+								$context->getLogger()->log(sprintf($str, $orderItem->getId(), $e->getMessage()));
152 152
 							}
153 153
 						}
154 154
 
155
-						$orderCount = count( $orderItems );
155
+						$orderCount = count($orderItems);
156 156
 						$orderStart += $orderCount;
157
-						$orderSearch->setSlice( $orderStart );
157
+						$orderSearch->setSlice($orderStart);
158 158
 					}
159
-					while( $orderCount >= $orderSearch->getSliceSize() );
159
+					while ($orderCount >= $orderSearch->getSliceSize());
160 160
 				}
161
-				catch( \Exception $e )
161
+				catch (\Exception $e)
162 162
 				{
163 163
 					$str = 'Error while capturing payments for service with ID "%1$s": %2$s';
164
-					$context->getLogger()->log( sprintf( $str, $serviceItem->getId(), $e->getMessage() ) );
164
+					$context->getLogger()->log(sprintf($str, $serviceItem->getId(), $e->getMessage()));
165 165
 				}
166 166
 			}
167 167
 
168
-			$count = count( $serviceItems );
168
+			$count = count($serviceItems);
169 169
 			$start += $count;
170
-			$serviceSearch->setSlice( $start );
170
+			$serviceSearch->setSlice($start);
171 171
 		}
172
-		while( $count >= $serviceSearch->getSliceSize() );
172
+		while ($count >= $serviceSearch->getSliceSize());
173 173
 	}
174 174
 }
Please login to merge, or discard this patch.
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -120,8 +120,7 @@  discard block
 block discarded – undo
120 120
 					{
121 121
 						$capdate = date( 'Y-m-d 00:00:00', time() - 86400 * $capDays );
122 122
 						$expr[] = $orderSearch->compare( '<=', 'order.datepayment', $capdate );
123
-					}
124
-					else
123
+					} else
125 124
 					{
126 125
 						$expr[] = $orderSearch->compare( '==', 'order.statusdelivery', $status );
127 126
 					}
@@ -144,8 +143,7 @@  discard block
 block discarded – undo
144 143
 							try
145 144
 							{
146 145
 								$serviceProvider->capture( $orderItem );
147
-							}
148
-							catch( \Exception $e )
146
+							} catch( \Exception $e )
149 147
 							{
150 148
 								$str = 'Error while capturing payment for order with ID "%1$s": %2$s';
151 149
 								$context->getLogger()->log( sprintf( $str, $orderItem->getId(), $e->getMessage() ) );
@@ -157,8 +155,7 @@  discard block
 block discarded – undo
157 155
 						$orderSearch->setSlice( $orderStart );
158 156
 					}
159 157
 					while( $orderCount >= $orderSearch->getSliceSize() );
160
-				}
161
-				catch( \Exception $e )
158
+				} catch( \Exception $e )
162 159
 				{
163 160
 					$str = 'Error while capturing payments for service with ID "%1$s": %2$s';
164 161
 					$context->getLogger()->log( sprintf( $str, $serviceItem->getId(), $e->getMessage() ) );
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Order/Service/Delivery/Factory.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string|null $name Name of the controller or "Standard" if null
31 31
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
32 32
 	 */
33
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
33
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
34 34
 	{
35 35
 		/** controller/jobs/order/service/delivery/name
36 36
 		 * Class name of the used order service delivery scheduler controller implementation
@@ -65,20 +65,20 @@  discard block
 block discarded – undo
65 65
 		 * @since 2014.03
66 66
 		 * @category Developer
67 67
 		 */
68
-		if( $name === null ) {
69
-			$name = $context->getConfig()->get( 'controller/jobs/order/service/delivery/name', 'Standard' );
68
+		if ($name === null) {
69
+			$name = $context->getConfig()->get('controller/jobs/order/service/delivery/name', 'Standard');
70 70
 		}
71 71
 
72
-		if( ctype_alnum( $name ) === false )
72
+		if (ctype_alnum($name) === false)
73 73
 		{
74
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Delivery\\' . $name : '<not a string>';
75
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
74
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Delivery\\' . $name : '<not a string>';
75
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
76 76
 		}
77 77
 
78 78
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
79 79
 		$classname = '\\Aimeos\\Controller\\Jobs\\Order\\Service\\Delivery\\' . $name;
80 80
 
81
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
81
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
82 82
 
83 83
 		/** controller/jobs/order/service/delivery/decorators/excludes
84 84
 		 * Excludes decorators added by the "common" option from the order service delivery controllers
@@ -154,6 +154,6 @@  discard block
 block discarded – undo
154 154
 		 * @see controller/jobs/order/service/delivery/decorators/excludes
155 155
 		 * @see controller/jobs/order/service/delivery/decorators/global
156 156
 		 */
157
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'order/service/delivery' );
157
+		return self::addControllerDecorators($context, $aimeos, $controller, 'order/service/delivery');
158 158
 	}
159 159
 }
160 160
\ No newline at end of file
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Order/Service/Delivery/Standard.php 2 patches
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	public function getName()
31 31
 	{
32
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Process order delivery services' );
32
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Process order delivery services');
33 33
 	}
34 34
 
35 35
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getDescription()
42 42
 	{
43
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Sends paid orders to the ERP system or logistic partner' );
43
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Sends paid orders to the ERP system or logistic partner');
44 44
 	}
45 45
 
46 46
 
@@ -68,75 +68,75 @@  discard block
 block discarded – undo
68 68
 		 * @see controller/jobs/order/email/payment/standard/limit-days
69 69
 		 * @see controller/jobs/order/email/delivery/standard/limit-days
70 70
 		 */
71
-		$days = $context->getConfig()->get( 'controller/jobs/order/service/delivery/limit-days', 90 );
72
-		$date = date( 'Y-m-d 00:00:00', time() - 86400 * $days );
71
+		$days = $context->getConfig()->get('controller/jobs/order/service/delivery/limit-days', 90);
72
+		$date = date('Y-m-d 00:00:00', time() - 86400 * $days);
73 73
 
74
-		$serviceManager = \Aimeos\MShop\Service\Manager\Factory::createManager( $context );
74
+		$serviceManager = \Aimeos\MShop\Service\Manager\Factory::createManager($context);
75 75
 		$serviceSearch = $serviceManager->createSearch();
76
-		$serviceSearch->setConditions( $serviceSearch->compare( '==', 'service.type.code', 'delivery' ) );
76
+		$serviceSearch->setConditions($serviceSearch->compare('==', 'service.type.code', 'delivery'));
77 77
 
78
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( $context );
78
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager($context);
79 79
 		$orderSearch = $orderManager->createSearch();
80 80
 
81 81
 		$start = 0;
82 82
 
83 83
 		do
84 84
 		{
85
-			$serviceItems = $serviceManager->searchItems( $serviceSearch );
85
+			$serviceItems = $serviceManager->searchItems($serviceSearch);
86 86
 
87
-			foreach( $serviceItems as $serviceItem )
87
+			foreach ($serviceItems as $serviceItem)
88 88
 			{
89 89
 				try
90 90
 				{
91
-					$serviceProvider = $serviceManager->getProvider( $serviceItem );
91
+					$serviceProvider = $serviceManager->getProvider($serviceItem);
92 92
 
93 93
 					$expr = array(
94
-						$orderSearch->compare( '==', 'order.siteid', $serviceItem->getSiteId() ),
95
-						$orderSearch->compare( '>', 'order.datepayment', $date ),
96
-						$orderSearch->compare( '>', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_PENDING ),
97
-						$orderSearch->compare( '==', 'order.statusdelivery', \Aimeos\MShop\Order\Item\Base::STAT_UNFINISHED ),
98
-						$orderSearch->compare( '==', 'order.base.service.code', $serviceItem->getCode() ),
99
-						$orderSearch->compare( '==', 'order.base.service.type', 'delivery' ),
94
+						$orderSearch->compare('==', 'order.siteid', $serviceItem->getSiteId()),
95
+						$orderSearch->compare('>', 'order.datepayment', $date),
96
+						$orderSearch->compare('>', 'order.statuspayment', \Aimeos\MShop\Order\Item\Base::PAY_PENDING),
97
+						$orderSearch->compare('==', 'order.statusdelivery', \Aimeos\MShop\Order\Item\Base::STAT_UNFINISHED),
98
+						$orderSearch->compare('==', 'order.base.service.code', $serviceItem->getCode()),
99
+						$orderSearch->compare('==', 'order.base.service.type', 'delivery'),
100 100
 					);
101
-					$orderSearch->setConditions( $orderSearch->combine( '&&', $expr ) );
101
+					$orderSearch->setConditions($orderSearch->combine('&&', $expr));
102 102
 
103 103
 					$orderStart = 0;
104 104
 
105 105
 					do
106 106
 					{
107
-						$orderItems = $orderManager->searchItems( $orderSearch );
107
+						$orderItems = $orderManager->searchItems($orderSearch);
108 108
 
109
-						foreach( $orderItems as $orderItem )
109
+						foreach ($orderItems as $orderItem)
110 110
 						{
111 111
 							try
112 112
 							{
113
-								$serviceProvider->process( $orderItem );
114
-								$orderManager->saveItem( $orderItem );
113
+								$serviceProvider->process($orderItem);
114
+								$orderManager->saveItem($orderItem);
115 115
 							}
116
-							catch( \Exception $e )
116
+							catch (\Exception $e)
117 117
 							{
118 118
 								$str = 'Error while processing order with ID "%1$s": %2$s';
119
-								$context->getLogger()->log( sprintf( $str, $orderItem->getId(), $e->getMessage() ) );
119
+								$context->getLogger()->log(sprintf($str, $orderItem->getId(), $e->getMessage()));
120 120
 							}
121 121
 						}
122 122
 
123
-						$orderCount = count( $orderItems );
123
+						$orderCount = count($orderItems);
124 124
 						$orderStart += $orderCount;
125
-						$orderSearch->setSlice( $orderStart );
125
+						$orderSearch->setSlice($orderStart);
126 126
 					}
127
-					while( $orderCount >= $orderSearch->getSliceSize() );
127
+					while ($orderCount >= $orderSearch->getSliceSize());
128 128
 				}
129
-				catch( \Exception $e )
129
+				catch (\Exception $e)
130 130
 				{
131 131
 					$str = 'Error while processing service with ID "%1$s": %2$s';
132
-					$context->getLogger()->log( sprintf( $str, $serviceItem->getId(), $e->getMessage() ) );
132
+					$context->getLogger()->log(sprintf($str, $serviceItem->getId(), $e->getMessage()));
133 133
 				}
134 134
 			}
135 135
 
136
-			$count = count( $serviceItems );
136
+			$count = count($serviceItems);
137 137
 			$start += $count;
138
-			$serviceSearch->setSlice( $start );
138
+			$serviceSearch->setSlice($start);
139 139
 		}
140
-		while( $count >= $serviceSearch->getSliceSize() );
140
+		while ($count >= $serviceSearch->getSliceSize());
141 141
 	}
142 142
 }
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -112,8 +112,7 @@  discard block
 block discarded – undo
112 112
 							{
113 113
 								$serviceProvider->process( $orderItem );
114 114
 								$orderManager->saveItem( $orderItem );
115
-							}
116
-							catch( \Exception $e )
115
+							} catch( \Exception $e )
117 116
 							{
118 117
 								$str = 'Error while processing order with ID "%1$s": %2$s';
119 118
 								$context->getLogger()->log( sprintf( $str, $orderItem->getId(), $e->getMessage() ) );
@@ -125,8 +124,7 @@  discard block
 block discarded – undo
125 124
 						$orderSearch->setSlice( $orderStart );
126 125
 					}
127 126
 					while( $orderCount >= $orderSearch->getSliceSize() );
128
-				}
129
-				catch( \Exception $e )
127
+				} catch( \Exception $e )
130 128
 				{
131 129
 					$str = 'Error while processing service with ID "%1$s": %2$s';
132 130
 					$context->getLogger()->log( sprintf( $str, $serviceItem->getId(), $e->getMessage() ) );
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Admin/Cache/Factory.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string|null $name Name of the controller or "Standard" if null
31 31
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
32 32
 	 */
33
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
33
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
34 34
 	{
35 35
 		/** controller/jobs/admin/cache/name
36 36
 		 * Class name of the used admin cache scheduler controller implementation
@@ -65,20 +65,20 @@  discard block
 block discarded – undo
65 65
 		 * @since 2014.03
66 66
 		 * @category Developer
67 67
 		 */
68
-		if( $name === null ) {
69
-			$name = $context->getConfig()->get( 'controller/jobs/admin/cache/name', 'Standard' );
68
+		if ($name === null) {
69
+			$name = $context->getConfig()->get('controller/jobs/admin/cache/name', 'Standard');
70 70
 		}
71 71
 
72
-		if( ctype_alnum( $name ) === false )
72
+		if (ctype_alnum($name) === false)
73 73
 		{
74
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Admin\\Cache\\' . $name : '<not a string>';
75
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
74
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Admin\\Cache\\' . $name : '<not a string>';
75
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
76 76
 		}
77 77
 
78 78
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
79 79
 		$classname = '\\Aimeos\\Controller\\Jobs\\Admin\\Cache\\' . $name;
80 80
 
81
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
81
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
82 82
 
83 83
 		/** controller/jobs/admin/cache/decorators/excludes
84 84
 		 * Excludes decorators added by the "common" option from the admin cache controllers
@@ -154,6 +154,6 @@  discard block
 block discarded – undo
154 154
 		 * @see controller/jobs/admin/cache/decorators/excludes
155 155
 		 * @see controller/jobs/admin/cache/decorators/global
156 156
 		 */
157
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'admin/cache' );
157
+		return self::addControllerDecorators($context, $aimeos, $controller, 'admin/cache');
158 158
 	}
159 159
 }
160 160
\ No newline at end of file
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Admin/Cache/Standard.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	public function getName()
31 31
 	{
32
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Cache cleanup' );
32
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Cache cleanup');
33 33
 	}
34 34
 
35 35
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getDescription()
42 42
 	{
43
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Removes the expired entries from the cache' );
43
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Removes the expired entries from the cache');
44 44
 	}
45 45
 
46 46
 
@@ -51,6 +51,6 @@  discard block
 block discarded – undo
51 51
 	 */
52 52
 	public function run()
53 53
 	{
54
-		\Aimeos\MAdmin\Cache\Manager\Factory::createManager( $this->getContext() )->getCache()->cleanup();
54
+		\Aimeos\MAdmin\Cache\Manager\Factory::createManager($this->getContext())->getCache()->cleanup();
55 55
 	}
56 56
 }
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Admin/Log/Factory.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string|null $name Name of the controller or "Standard" if null
31 31
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
32 32
 	 */
33
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
33
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
34 34
 	{
35 35
 		/** controller/jobs/admin/log/name
36 36
 		 * Class name of the used admin log scheduler controller implementation
@@ -65,20 +65,20 @@  discard block
 block discarded – undo
65 65
 		 * @since 2014.09
66 66
 		 * @category Developer
67 67
 		 */
68
-		if( $name === null ) {
69
-			$name = $context->getConfig()->get( 'controller/jobs/admin/log/name', 'Standard' );
68
+		if ($name === null) {
69
+			$name = $context->getConfig()->get('controller/jobs/admin/log/name', 'Standard');
70 70
 		}
71 71
 
72
-		if( ctype_alnum( $name ) === false )
72
+		if (ctype_alnum($name) === false)
73 73
 		{
74
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Admin\\Log\\' . $name : '<not a string>';
75
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
74
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Admin\\Log\\' . $name : '<not a string>';
75
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
76 76
 		}
77 77
 
78 78
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
79 79
 		$classname = '\\Aimeos\\Controller\\Jobs\\Admin\\Log\\' . $name;
80 80
 
81
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
81
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
82 82
 
83 83
 		/** controller/jobs/admin/log/decorators/excludes
84 84
 		 * Excludes decorators added by the "common" option from the admin log controllers
@@ -154,6 +154,6 @@  discard block
 block discarded – undo
154 154
 		 * @see controller/jobs/admin/log/decorators/excludes
155 155
 		 * @see controller/jobs/admin/log/decorators/global
156 156
 		 */
157
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'admin/log' );
157
+		return self::addControllerDecorators($context, $aimeos, $controller, 'admin/log');
158 158
 	}
159 159
 }
160 160
\ No newline at end of file
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Admin/Log/Standard.php 1 patch
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 */
30 30
 	public function getName()
31 31
 	{
32
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Log cleanup' );
32
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Log cleanup');
33 33
 	}
34 34
 
35 35
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public function getDescription()
42 42
 	{
43
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Removes the old log entries from the database and archives them (optional)' );
43
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Removes the old log entries from the database and archives them (optional)');
44 44
 	}
45 45
 
46 46
 
@@ -71,8 +71,8 @@  discard block
 block discarded – undo
71 71
 		 * @see controller/jobs/admin/log/standard/container/format
72 72
 		 * @see controller/jobs/admin/log/standard/container/options
73 73
 		 */
74
-		$limit = $config->get( 'controller/jobs/admin/log/standard/limit-days', 30 );
75
-		$limitDate = date( 'Y-m-d H:i:s', time() - $limit * 86400 );
74
+		$limit = $config->get('controller/jobs/admin/log/standard/limit-days', 30);
75
+		$limitDate = date('Y-m-d H:i:s', time() - $limit * 86400);
76 76
 
77 77
 		/** controller/jobs/admin/log/standard/path
78 78
 		 * Path to a writable directory where the log archive files should be stored
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 		 * @see controller/jobs/admin/log/standard/container/options
102 102
 		 * @see controller/jobs/admin/log/standard/limit-days
103 103
 		 */
104
-		if( ( $path = $config->get( 'controller/jobs/admin/log/standard/path', null ) ) !== null )
104
+		if (($path = $config->get('controller/jobs/admin/log/standard/path', null)) !== null)
105 105
 		{
106 106
 			/** controller/jobs/admin/log/standard/container/type
107 107
 			 * Container file type storing all coupon code files to import
@@ -175,19 +175,19 @@  discard block
 block discarded – undo
175 175
 			 * @see controller/jobs/admin/log/standard/limit-days
176 176
 			 */
177 177
 
178
-			$type = $config->get( 'controller/jobs/admin/log/standard/container/type', 'Zip' );
179
-			$format = $config->get( 'controller/jobs/admin/log/standard/container/format', 'CSV' );
180
-			$options = $config->get( 'controller/jobs/admin/log/standard/container/options', array() );
178
+			$type = $config->get('controller/jobs/admin/log/standard/container/type', 'Zip');
179
+			$format = $config->get('controller/jobs/admin/log/standard/container/format', 'CSV');
180
+			$options = $config->get('controller/jobs/admin/log/standard/container/options', array());
181 181
 
182
-			$path .= DIRECTORY_SEPARATOR . str_replace( ' ', '_', $limitDate );
183
-			$container = \Aimeos\MW\Container\Factory::getContainer( $path, $type, $format, $options );
182
+			$path .= DIRECTORY_SEPARATOR . str_replace(' ', '_', $limitDate);
183
+			$container = \Aimeos\MW\Container\Factory::getContainer($path, $type, $format, $options);
184 184
 		}
185 185
 
186
-		$manager = \Aimeos\MAdmin\Factory::createManager( $context, 'log' );
186
+		$manager = \Aimeos\MAdmin\Factory::createManager($context, 'log');
187 187
 
188 188
 		$search = $manager->createSearch();
189
-		$search->setConditions( $search->compare( '<=', 'log.timestamp', $limitDate ) );
190
-		$search->setSortations( array( $search->sort( '+', 'log.timestamp' ) ) );
189
+		$search->setConditions($search->compare('<=', 'log.timestamp', $limitDate));
190
+		$search->setSortations(array($search->sort('+', 'log.timestamp')));
191 191
 
192 192
 		$start = 0;
193 193
 		$contents = array();
@@ -195,37 +195,37 @@  discard block
 block discarded – undo
195 195
 		do
196 196
 		{
197 197
 			$ids = array();
198
-			$items = $manager->searchItems( $search );
198
+			$items = $manager->searchItems($search);
199 199
 
200
-			foreach( $items as $id => $item )
200
+			foreach ($items as $id => $item)
201 201
 			{
202
-				if( $container !== null )
202
+				if ($container !== null)
203 203
 				{
204 204
 					$facility = $item->getFacility();
205 205
 
206
-					if( !isset( $contents[$facility] ) ) {
207
-						$contents[$facility] = $container->create( $facility );
206
+					if (!isset($contents[$facility])) {
207
+						$contents[$facility] = $container->create($facility);
208 208
 					}
209 209
 
210
-					$contents[$facility]->add( $item->toArray() );
210
+					$contents[$facility]->add($item->toArray());
211 211
 				}
212 212
 
213 213
 				$ids[] = $id;
214 214
 			}
215 215
 
216
-			$manager->deleteItems( $ids );
216
+			$manager->deleteItems($ids);
217 217
 
218
-			$count = count( $items );
218
+			$count = count($items);
219 219
 			$start += $count;
220
-			$search->setSlice( $start );
220
+			$search->setSlice($start);
221 221
 		}
222
-		while( $count >= $search->getSliceSize() );
222
+		while ($count >= $search->getSliceSize());
223 223
 
224 224
 
225
-		if( $container !== null && !empty( $contents ) )
225
+		if ($container !== null && !empty($contents))
226 226
 		{
227
-			foreach( $contents as $content ) {
228
-				$container->add( $content );
227
+			foreach ($contents as $content) {
228
+				$container->add($content);
229 229
 			}
230 230
 
231 231
 			$container->close();
Please login to merge, or discard this patch.
controller/jobs/src/Controller/Jobs/Index/Optimize/Factory.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string|null $name Name of the controller or "Standard" if null
31 31
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
32 32
 	 */
33
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
33
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
34 34
 	{
35 35
 		/** controller/jobs/index/optimize/name
36 36
 		 * Class name of the used catalog index optimize scheduler controller implementation
@@ -65,20 +65,20 @@  discard block
 block discarded – undo
65 65
 		 * @since 2014.03
66 66
 		 * @category Developer
67 67
 		 */
68
-		if( $name === null ) {
69
-			$name = $context->getConfig()->get( 'controller/jobs/index/optimize/name', 'Standard' );
68
+		if ($name === null) {
69
+			$name = $context->getConfig()->get('controller/jobs/index/optimize/name', 'Standard');
70 70
 		}
71 71
 
72
-		if( ctype_alnum( $name ) === false )
72
+		if (ctype_alnum($name) === false)
73 73
 		{
74
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Index\\Optimize\\' . $name : '<not a string>';
75
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
74
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Index\\Optimize\\' . $name : '<not a string>';
75
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
76 76
 		}
77 77
 
78 78
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
79 79
 		$classname = '\\Aimeos\\Controller\\Jobs\\Index\\Optimize\\' . $name;
80 80
 
81
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
81
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
82 82
 
83 83
 		/** controller/jobs/index/optimize/decorators/excludes
84 84
 		 * Excludes decorators added by the "common" option from the catalog index optimize controllers
@@ -154,6 +154,6 @@  discard block
 block discarded – undo
154 154
 		 * @see controller/jobs/index/optimize/decorators/excludes
155 155
 		 * @see controller/jobs/index/optimize/decorators/global
156 156
 		 */
157
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'index/optimize' );
157
+		return self::addControllerDecorators($context, $aimeos, $controller, 'index/optimize');
158 158
 	}
159 159
 }
160 160
\ No newline at end of file
Please login to merge, or discard this patch.