Completed
Push — master ( c7cb6f...a11e92 )
by Aimeos
10:48
created
lib/mshoplib/src/MShop/Plugin/Manager/Base.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 		}
47 47
 
48 48
 		$interface = '\\Aimeos\\MShop\\Plugin\\Provider\\Factory\\Iface';
49
-		$classname = '\\Aimeos\\MShop\\Plugin\\Provider\\' . $type . '\\' . $provider;
49
+		$classname = '\\Aimeos\\MShop\\Plugin\\Provider\\'.$type.'\\'.$provider;
50 50
 
51 51
 		if( class_exists( $classname ) === false ) {
52 52
 			throw new \Aimeos\MShop\Plugin\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		 * @category Developer
85 85
 		 * @see mshop/plugin/provider/order/decorators
86 86
 		 */
87
-		$decorators = $config->get( 'mshop/plugin/provider/' . $item->getType() . '/decorators', [] );
87
+		$decorators = $config->get( 'mshop/plugin/provider/'.$item->getType().'/decorators', [] );
88 88
 
89 89
 		$provider = $this->addPluginDecorators( $item, $provider, $names );
90 90
 		$provider = $this->addPluginDecorators( $item, $provider, $decorators );
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 				throw new \Aimeos\MShop\Plugin\Exception( sprintf( 'Invalid characters in class name "%1$s"', $name ) );
113 113
 			}
114 114
 
115
-			$classname = $classprefix . $name;
115
+			$classname = $classprefix.$name;
116 116
 
117 117
 			if( class_exists( $classname ) === false ) {
118 118
 				throw new \Aimeos\MShop\Plugin\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Service/Manager/Base.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 		}
47 47
 
48 48
 		$interface = '\\Aimeos\\MShop\\Service\\Provider\\Factory\\Iface';
49
-		$classname = '\\Aimeos\\MShop\\Service\\Provider\\' . $type . '\\' . $provider;
49
+		$classname = '\\Aimeos\\MShop\\Service\\Provider\\'.$type.'\\'.$provider;
50 50
 
51 51
 		if( class_exists( $classname ) === false ) {
52 52
 			throw new \Aimeos\MShop\Service\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 		 * @category Developer
108 108
 		 * @see mshop/service/provider/delivery/decorators
109 109
 		 */
110
-		$decorators = $config->get( 'mshop/service/provider/' . $item->getType() . '/decorators', [] );
110
+		$decorators = $config->get( 'mshop/service/provider/'.$item->getType().'/decorators', [] );
111 111
 
112 112
 		$provider = $this->addServiceDecorators( $item, $provider, $names );
113 113
 		return $this->addServiceDecorators( $item, $provider, $decorators );
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 				throw new \Aimeos\MShop\Service\Exception( sprintf( 'Invalid characters in class name "%1$s"', $name ) );
135 135
 			}
136 136
 
137
-			$classname = $classprefix . $name;
137
+			$classname = $classprefix.$name;
138 138
 
139 139
 			if( class_exists( $classname ) === false ) {
140 140
 				throw new \Aimeos\MShop\Service\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
Please login to merge, or discard this patch.
lib/mshoplib/tests/MShop/Coupon/Provider/Decorator/OnceTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 	public function testIsAvailable()
44 44
 	{
45 45
 		$provider = new \Aimeos\MShop\Coupon\Provider\Example( $this->context, $this->couponItem, 'ABCD' );
46
-		$object = new \Aimeos\MShop\Coupon\Provider\Decorator\Once( $provider, $this->context, $this->couponItem, 'ABCD');
46
+		$object = new \Aimeos\MShop\Coupon\Provider\Decorator\Once( $provider, $this->context, $this->couponItem, 'ABCD' );
47 47
 		$object->setObject( $object );
48 48
 
49 49
 		$this->assertTrue( $object->isAvailable( $this->orderBase ) );
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 	public function testIsAvailableExisting()
54 54
 	{
55 55
 		$provider = new \Aimeos\MShop\Coupon\Provider\Example( $this->context, $this->couponItem, 'OPQR' );
56
-		$object = new \Aimeos\MShop\Coupon\Provider\Decorator\Once( $provider, $this->context, $this->couponItem, 'OPQR');
56
+		$object = new \Aimeos\MShop\Coupon\Provider\Decorator\Once( $provider, $this->context, $this->couponItem, 'OPQR' );
57 57
 		$object->setObject( $object );
58 58
 
59 59
 		$this->assertFalse( $object->isAvailable( $this->orderBase ) );
Please login to merge, or discard this patch.
lib/mshoplib/tests/MShop/Coupon/Provider/Decorator/SupplierTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 		$this->couponItem = \Aimeos\MShop\Coupon\Manager\Factory::createManager( $this->context )->createItem();
25 25
 
26 26
 		$provider = new \Aimeos\MShop\Coupon\Provider\Example( $this->context, $this->couponItem, 'abcd' );
27
-		$this->object = new \Aimeos\MShop\Coupon\Provider\Decorator\Supplier( $provider, $this->context, $this->couponItem, 'abcd');
27
+		$this->object = new \Aimeos\MShop\Coupon\Provider\Decorator\Supplier( $provider, $this->context, $this->couponItem, 'abcd' );
28 28
 		$this->object->setObject( $this->object );
29 29
 
30 30
 		$priceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'price' );
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Catalog/Item/Standard.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 	 * Sets a new URL target specific for that category
234 234
 	 *
235 235
 	 * @param string $value New URL target specific for that category
236
-	 * @return \Aimeos\MShop\Product\Item\Iface Product item for chaining method calls
236
+	 * @return Standard Product item for chaining method calls
237 237
 	 */
238 238
 	public function setTarget( $value )
239 239
 	{
@@ -366,7 +366,7 @@  discard block
 block discarded – undo
366 366
 	/**
367 367
 	 * Returns all children of this node.
368 368
 	 *
369
-	 * @return array Numerically indexed list of nodes
369
+	 * @return \Aimeos\MShop\Common\Item\Tree\Iface[] Numerically indexed list of nodes
370 370
 	 */
371 371
 	public function getChildren()
372 372
 	{
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Locale/Item/Site/Standard.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	 * Initializes the site object.
30 30
 	 *
31 31
 	 * @param array $values Associative list of item key/value pairs
32
-	 * @param array $children List of nodes implementing \Aimeos\MW\Tree\Node\Iface
32
+	 * @param Standard[] $children List of nodes implementing \Aimeos\MW\Tree\Node\Iface
33 33
 	 */
34 34
 	public function __construct( array $values = [], array $children = [] )
35 35
 	{
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	/**
161 161
 	 * Returns the ID of the parent site
162 162
 	 *
163
-	 * @return string Unique ID of the parent site
163
+	 * @return integer Unique ID of the parent site
164 164
 	 */
165 165
 	public function getParentId()
166 166
 	{
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Order/Item/Base/Coupon/Standard.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -122,7 +122,7 @@
 block discarded – undo
122 122
 	{
123 123
 		if( (string) $code !== $this->getCode() )
124 124
 		{
125
-			$this->values['order.base.coupon.code'] = (string) $this->checkCode( $code );;
125
+			$this->values['order.base.coupon.code'] = (string) $this->checkCode( $code ); ;
126 126
 			$this->setModified();
127 127
 		}
128 128
 
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Common/Item/Base.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	 */
78 78
 	public function getId()
79 79
 	{
80
-		$key = $this->prefix . 'id';
80
+		$key = $this->prefix.'id';
81 81
 
82 82
 		if( isset( $this->bdata[$key] ) && $this->bdata[$key] != '' ) {
83 83
 			return (string) $this->bdata[$key];
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function setId( $id )
99 99
 	{
100
-		$key = $this->prefix . 'id';
100
+		$key = $this->prefix.'id';
101 101
 
102 102
 		if( ( $this->bdata[$key] = \Aimeos\MShop\Common\Item\Base::checkId( $this->getId(), $id ) ) === null ) {
103 103
 			$this->modified = true;
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 	 */
118 118
 	public function getSiteId()
119 119
 	{
120
-		$key = $this->prefix . 'siteid';
120
+		$key = $this->prefix.'siteid';
121 121
 
122 122
 		if( isset( $this->bdata[$key] ) ) {
123 123
 			return (string) $this->bdata[$key];
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
 	 */
137 137
 	public function getTimeModified()
138 138
 	{
139
-		$key = $this->prefix . 'mtime';
139
+		$key = $this->prefix.'mtime';
140 140
 
141 141
 		if( isset( $this->bdata[$key] ) ) {
142 142
 			return (string) $this->bdata[$key];
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 	 */
156 156
 	public function getTimeCreated()
157 157
 	{
158
-		$key = $this->prefix . 'ctime';
158
+		$key = $this->prefix.'ctime';
159 159
 
160 160
 		if( isset( $this->bdata[$key] ) ) {
161 161
 			return (string) $this->bdata[$key];
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
 	 */
175 175
 	public function getEditor()
176 176
 	{
177
-		$key = $this->prefix . 'editor';
177
+		$key = $this->prefix.'editor';
178 178
 
179 179
 		if( isset( $this->bdata[$key] ) ) {
180 180
 			return (string) $this->bdata[$key];
@@ -219,16 +219,16 @@  discard block
 block discarded – undo
219 219
 	 */
220 220
 	public function fromArray( array $list )
221 221
 	{
222
-		if( array_key_exists( $this->prefix . 'id', $list ) )
222
+		if( array_key_exists( $this->prefix.'id', $list ) )
223 223
 		{
224
-			$this->setId( $list[$this->prefix . 'id'] );
225
-			unset( $list[$this->prefix . 'id'] );
224
+			$this->setId( $list[$this->prefix.'id'] );
225
+			unset( $list[$this->prefix.'id'] );
226 226
 		}
227 227
 
228
-		unset( $list[$this->prefix . 'siteid'] );
229
-		unset( $list[$this->prefix . 'ctime'] );
230
-		unset( $list[$this->prefix . 'mtime'] );
231
-		unset( $list[$this->prefix . 'editor'] );
228
+		unset( $list[$this->prefix.'siteid'] );
229
+		unset( $list[$this->prefix.'ctime'] );
230
+		unset( $list[$this->prefix.'mtime'] );
231
+		unset( $list[$this->prefix.'editor'] );
232 232
 
233 233
 		return $list;
234 234
 	}
@@ -242,14 +242,14 @@  discard block
 block discarded – undo
242 242
 	 */
243 243
 	public function toArray( $private = false )
244 244
 	{
245
-		$list = [$this->prefix . 'id' => $this->getId()];
245
+		$list = [$this->prefix.'id' => $this->getId()];
246 246
 
247 247
 		if( $private === true )
248 248
 		{
249
-			$list[$this->prefix . 'siteid'] = $this->getSiteId();
250
-			$list[$this->prefix . 'ctime'] = $this->getTimeCreated();
251
-			$list[$this->prefix . 'mtime'] = $this->getTimeModified();
252
-			$list[$this->prefix . 'editor'] = $this->getEditor();
249
+			$list[$this->prefix.'siteid'] = $this->getSiteId();
250
+			$list[$this->prefix.'ctime'] = $this->getTimeCreated();
251
+			$list[$this->prefix.'mtime'] = $this->getTimeModified();
252
+			$list[$this->prefix.'editor'] = $this->getEditor();
253 253
 		}
254 254
 
255 255
 		return $list;
Please login to merge, or discard this patch.
lib/mshoplib/src/MShop/Common/Item/Address/Standard.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -46,8 +46,8 @@  discard block
 block discarded – undo
46 46
 	 */
47 47
 	public function getParentId()
48 48
 	{
49
-		if( isset( $this->values[$this->prefix . 'parentid'] ) ) {
50
-			return (string) $this->values[$this->prefix . 'parentid'];
49
+		if( isset( $this->values[$this->prefix.'parentid'] ) ) {
50
+			return (string) $this->values[$this->prefix.'parentid'];
51 51
 		}
52 52
 
53 53
 		return '';
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 	{
65 65
 		if( (string) $parentid !== $this->getParentId() )
66 66
 		{
67
-			$this->values[$this->prefix . 'parentid'] = (string) $parentid;
67
+			$this->values[$this->prefix.'parentid'] = (string) $parentid;
68 68
 			$this->setModified();
69 69
 		}
70 70
 
@@ -79,8 +79,8 @@  discard block
 block discarded – undo
79 79
 	 */
80 80
 	public function getPosition()
81 81
 	{
82
-		if( isset( $this->values[$this->prefix . 'position'] ) ) {
83
-			return (int) $this->values[$this->prefix . 'position'];
82
+		if( isset( $this->values[$this->prefix.'position'] ) ) {
83
+			return (int) $this->values[$this->prefix.'position'];
84 84
 		}
85 85
 
86 86
 		return 0;
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	{
98 98
 		if( (int) $position !== $this->getPosition() )
99 99
 		{
100
-			$this->values[$this->prefix . 'position'] = (int) $position;
100
+			$this->values[$this->prefix.'position'] = (int) $position;
101 101
 			$this->setModified();
102 102
 		}
103 103
 
@@ -120,8 +120,8 @@  discard block
 block discarded – undo
120 120
 		{
121 121
 			switch( $key )
122 122
 			{
123
-				case $this->prefix . 'parentid': $this->setParentId( $value ); break;
124
-				case $this->prefix . 'position': $this->setPosition( $value ); break;
123
+				case $this->prefix.'parentid': $this->setParentId( $value ); break;
124
+				case $this->prefix.'position': $this->setPosition( $value ); break;
125 125
 				default: $unknown[$key] = $value;
126 126
 			}
127 127
 		}
@@ -140,10 +140,10 @@  discard block
 block discarded – undo
140 140
 	{
141 141
 		$list = parent::toArray( $private );
142 142
 
143
-		$list[$this->prefix . 'position'] = $this->getPosition();
143
+		$list[$this->prefix.'position'] = $this->getPosition();
144 144
 
145 145
 		if( $private === true ) {
146
-			$list[$this->prefix . 'parentid'] = $this->getParentId();
146
+			$list[$this->prefix.'parentid'] = $this->getParentId();
147 147
 		}
148 148
 
149 149
 		return $list;
Please login to merge, or discard this patch.