Inspection of "Prevent add button to be on top after moving/resor..." - aimeos/ai-admin-jqadm - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 8248a2...199ce1 )

by Aimeos

created 2019-09-04 07:11 UTC

Status

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -32,14 +32,14 @@  discard block
 block discarded – undo
 					v-bind:aria-controls="'item-address-group-data-' + idx" aria-expanded="false">
 					<div class="card-tools-left">
 						<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
+							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
 						</div>
 					</div>
 					<span class="item-label header-label">{{ getLabel(idx, 'customer.address.') }}</span>
 					&nbsp;
 					<div class="card-tools-right">
 						<div class="btn btn-card-header act-copy fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Duplicate entry (Ctrl+D)') ); ?>"
+							title="<?= $enc->attr( $this->translate( 'admin', 'Duplicate entry (Ctrl+D)' ) ); ?>"
 							v-on:click.stop="duplicateItem(idx)">
 						</div>
 						<div v-if="!checkSite('customer.address.siteid', idx)"
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
 						</div>
 						<div v-if="!checkSite('customer.address.siteid', idx)"
 							class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
+							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
 							v-on:click="removeItem(idx)">
 						</div>
 					</div>
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
 
 			<div slot="footer" class="card-tools-more">
 				<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
 					v-on:click="addItem('customer.address.')" >
 				</div>
 			</div>

Please login to merge, or discard this patch.

admin/jqadm/templates/service/item-text-standard.php 1 patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -30,7 +30,7 @@  discard block
 block discarded – undo
 					v-bind:aria-controls="'item-text-group-data-' + idx" aria-expanded="false">
 					<div class="card-tools-left">
 						<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
+							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
 						</div>
 					</div>
 					<span class="item-label header-label" v-html="getLabel(idx)"></span>
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
 						</div>
 						<div v-if="!checkSite('service.lists.siteid', idx)"
 							class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
+							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
 							v-on:click.stop="removeItem(idx)">
 						</div>
 					</div>
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
 					<div v-on:click="toggle(idx)" class="col-xl-12 advanced" v-bind:class="{ 'collapsed': !advanced[idx] }">
 						<div class="card-tools-left">
 							<div class="btn act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data') ); ?>">
+								title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data' ) ); ?>">
 							</div>
 						</div>
 						<span class="header-label"><?= $enc->html( $this->translate( 'admin', 'Advanced' ) ); ?></span>
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
 									</th>
 									<th class="actions">
 										<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-											title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+											title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
 											v-bind:readonly="checkSite('service.lists.siteid', idx)"
 											v-on:click="addConfig(idx)" >
 										</div>
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
 									<td class="actions">
 										<div v-if="!checkSite('service.lists.siteid', idx)" v-on:click="removeConfig(idx, pos)"
 											class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-											title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
+											title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>">
 										</div>
 									</td>
 								</tr>
@@ -305,7 +305,7 @@  discard block
 block discarded – undo
 
 			<div slot="footer" class="card-tools-more">
 				<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
 					v-on:click="addItem('service.lists.')" >
 				</div>
 			</div>

Please login to merge, or discard this patch.

admin/jqadm/templates/attribute/item-media-standard.php 1 patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -30,7 +30,7 @@  discard block
 block discarded – undo
 					v-bind:aria-controls="'item-media-group-data-' + idx" aria-expanded="false">
 					<div class="card-tools-left">
 						<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
+							title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
 						</div>
 					</div>
 					<span class="item-label header-label" v-html="getLabel(idx)"></span>
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
 						</div>
 						<div v-if="!checkSite('attribute.lists.siteid', idx)"
 							class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
+							title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
 							v-on:click.stop="removeItem(idx)">
 						</div>
 					</div>
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
 								v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'media.url' ) ) ); ?>'.replace( 'idx', idx )"
 								v-model="items[idx]['media.url']" />
 							<img v-if="items[idx]['media.preview']" class="item-preview"
-								v-bind:src="getUrl('<?= $this->content('') ?>', items[idx]['media.preview'])"
+								v-bind:src="getUrl('<?= $this->content( '' ) ?>', items[idx]['media.preview'])"
 								v-bind:alt="items[idx]['media.label']" />
 							<p v-else class="item-preview">
 								{{ items[idx]['media.label'] || '<?= $enc->html( $this->translate( 'admin', 'Select file' ) ) ?>' }}
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
 					<div v-on:click="toggle(idx)" class="col-xl-12 advanced" v-bind:class="{ 'collapsed': !advanced[idx] }">
 						<div class="card-tools-left">
 							<div class="btn act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data') ); ?>">
+								title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data' ) ); ?>">
 							</div>
 						</div>
 						<span class="header-label"><?= $enc->html( $this->translate( 'admin', 'Advanced' ) ); ?></span>
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
 									</th>
 									<th class="actions">
 										<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-											title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+											title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
 											v-bind:readonly="checkSite('attribute.lists.siteid', idx)"
 											v-on:click="addConfig(idx)" >
 										</div>
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
 									<td class="actions">
 										<div v-if="!checkSite('attribute.lists.siteid', idx)" v-on:click="removeConfig(idx, pos)"
 											class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-											title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
+											title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>">
 										</div>
 									</td>
 								</tr>
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
 
 			<div slot="footer" class="card-tools-more">
 				<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+					title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
 					v-on:click="addItem('attribute.lists.')" >
 				</div>
 			</div>

Please login to merge, or discard this patch.

		@@ -32,14 +32,14 @@ discard block
		block discarded – undo
32	32	v-bind:aria-controls="'item-address-group-data-' + idx" aria-expanded="false">
33	33	<div class="card-tools-left">
34	34	<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
35		- title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
	35	+ title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
36	36	</div>
37	37	</div>
38	38	<span class="item-label header-label">{{ getLabel(idx, 'customer.address.') }}</span>
39	39
40	40	<div class="card-tools-right">
41	41	<div class="btn btn-card-header act-copy fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
42		- title="<?= $enc->attr( $this->translate( 'admin', 'Duplicate entry (Ctrl+D)') ); ?>"
	42	+ title="<?= $enc->attr( $this->translate( 'admin', 'Duplicate entry (Ctrl+D)' ) ); ?>"
43	43	v-on:click.stop="duplicateItem(idx)">
44	44	</div>
45	45	<div v-if="!checkSite('customer.address.siteid', idx)"
		@@ -48,7 +48,7 @@ discard block
		block discarded – undo
48	48	</div>
49	49	<div v-if="!checkSite('customer.address.siteid', idx)"
50	50	class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
51		- title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
	51	+ title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
52	52	v-on:click="removeItem(idx)">
53	53	</div>
54	54	</div>
		@@ -342,7 +342,7 @@ discard block
		block discarded – undo
342	342
343	343	<div slot="footer" class="card-tools-more">
344	344	<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
345		- title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
	345	+ title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
346	346	v-on:click="addItem('customer.address.')" >
347	347	</div>
348	348	</div>

		@@ -30,7 +30,7 @@ discard block
		block discarded – undo
30	30	v-bind:aria-controls="'item-text-group-data-' + idx" aria-expanded="false">
31	31	<div class="card-tools-left">
32	32	<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
33		- title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
	33	+ title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
34	34	</div>
35	35	</div>
36	36	<span class="item-label header-label" v-html="getLabel(idx)"></span>
		@@ -42,7 +42,7 @@ discard block
		block discarded – undo
42	42	</div>
43	43	<div v-if="!checkSite('service.lists.siteid', idx)"
44	44	class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
45		- title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
	45	+ title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
46	46	v-on:click.stop="removeItem(idx)">
47	47	</div>
48	48	</div>
		@@ -180,7 +180,7 @@ discard block
		block discarded – undo
180	180	<div v-on:click="toggle(idx)" class="col-xl-12 advanced" v-bind:class="{ 'collapsed': !advanced[idx] }">
181	181	<div class="card-tools-left">
182	182	<div class="btn act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
183		- title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data') ); ?>">
	183	+ title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data' ) ); ?>">
184	184	</div>
185	185	</div>
186	186	<span class="header-label"><?= $enc->html( $this->translate( 'admin', 'Advanced' ) ); ?></span>
		@@ -262,7 +262,7 @@ discard block
		block discarded – undo
262	262	</th>
263	263	<th class="actions">
264	264	<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
265		- title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
	265	+ title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
266	266	v-bind:readonly="checkSite('service.lists.siteid', idx)"
267	267	v-on:click="addConfig(idx)" >
268	268	</div>
		@@ -289,7 +289,7 @@ discard block
		block discarded – undo
289	289	<td class="actions">
290	290	<div v-if="!checkSite('service.lists.siteid', idx)" v-on:click="removeConfig(idx, pos)"
291	291	class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
292		- title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
	292	+ title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>">
293	293	</div>
294	294	</td>
295	295	</tr>
		@@ -305,7 +305,7 @@ discard block
		block discarded – undo
305	305
306	306	<div slot="footer" class="card-tools-more">
307	307	<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
308		- title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
	308	+ title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
309	309	v-on:click="addItem('service.lists.')" >
310	310	</div>
311	311	</div>

		@@ -30,7 +30,7 @@ discard block
		block discarded – undo
30	30	v-bind:aria-controls="'item-media-group-data-' + idx" aria-expanded="false">
31	31	<div class="card-tools-left">
32	32	<div class="btn btn-card-header act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
33		- title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry') ); ?>">
	33	+ title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide this entry' ) ); ?>">
34	34	</div>
35	35	</div>
36	36	<span class="item-label header-label" v-html="getLabel(idx)"></span>
		@@ -42,7 +42,7 @@ discard block
		block discarded – undo
42	42	</div>
43	43	<div v-if="!checkSite('attribute.lists.siteid', idx)"
44	44	class="btn btn-card-header act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
45		- title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
	45	+ title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>"
46	46	v-on:click.stop="removeItem(idx)">
47	47	</div>
48	48	</div>
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'media.url' ) ) ); ?>'.replace( 'idx', idx )"
65	65	v-model="items[idx]['media.url']" />
66	66	<img v-if="items[idx]['media.preview']" class="item-preview"
67		- v-bind:src="getUrl('<?= $this->content('') ?>', items[idx]['media.preview'])"
	67	+ v-bind:src="getUrl('<?= $this->content( '' ) ?>', items[idx]['media.preview'])"
68	68	v-bind:alt="items[idx]['media.label']" />
69	69	<p v-else class="item-preview">
70	70	{{ items[idx]['media.label'] \|\| '<?= $enc->html( $this->translate( 'admin', 'Select file' ) ) ?>' }}
		@@ -173,7 +173,7 @@ discard block
		block discarded – undo
173	173	<div v-on:click="toggle(idx)" class="col-xl-12 advanced" v-bind:class="{ 'collapsed': !advanced[idx] }">
174	174	<div class="card-tools-left">
175	175	<div class="btn act-show fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
176		- title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data') ); ?>">
	176	+ title="<?= $enc->attr( $this->translate( 'admin', 'Show/hide advanced data' ) ); ?>">
177	177	</div>
178	178	</div>
179	179	<span class="header-label"><?= $enc->html( $this->translate( 'admin', 'Advanced' ) ); ?></span>
		@@ -252,7 +252,7 @@ discard block
		block discarded – undo
252	252	</th>
253	253	<th class="actions">
254	254	<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
255		- title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
	255	+ title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
256	256	v-bind:readonly="checkSite('attribute.lists.siteid', idx)"
257	257	v-on:click="addConfig(idx)" >
258	258	</div>
		@@ -279,7 +279,7 @@ discard block
		block discarded – undo
279	279	<td class="actions">
280	280	<div v-if="!checkSite('attribute.lists.siteid', idx)" v-on:click="removeConfig(idx, pos)"
281	281	class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
282		- title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
	282	+ title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry' ) ); ?>">
283	283	</div>
284	284	</td>
285	285	</tr>
		@@ -295,7 +295,7 @@ discard block
		block discarded – undo
295	295
296	296	<div slot="footer" class="card-tools-more">
297	297	<div class="btn btn-primary btn-card-more act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
298		- title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
	298	+ title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)' ) ); ?>"
299	299	v-on:click="addItem('attribute.lists.')" >
300	300	</div>
301	301	</div>

aimeos / ai-admin-jqadm

Push — master ( 8248a2...199ce1 )

Status

Category

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch