ContentBlock::fieldLabels() - Code Metrics - Inspection of "Merge pull request #105 from dhensby/pulls/content..." - sheadawson/silverstripe-blocks - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a4eef9...fb22a5 )

by Shea

created 2016-10-30 21:40 UTC

ContentBlock::fieldLabels() A

↳ Parent: ContentBlock

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	1

<?php

class ContentBlock extends Block
namespace YourVendor;

class YourClass { }
{
    /**
     * If the singular name is set in a private static $singular_name, it cannot be changed using the translation files
     * for some reason. Fix it by defining a method that handles the translation.
     * @return string
     */
    public function singular_name()
    {
        return _t('ContentBlock.SINGULARNAME', 'Content Block');
    }

    /**
     * If the plural name is set in a private static $plural_name, it cannot be changed using the translation files
     * for some reason. Fix it by defining a method that handles the translation.
     * @return string
     */
    public function plural_name()
    {
        return _t('ContentBlock.PLURALNAME', 'Content Blocks');
    }

    private static $db = array(

        'Content' => 'HTMLText',
    );

	public function fieldLabels($includeRelations = true)
	{
		return array_merge(
			parent::fieldLabels($includeRelations),
			array(
				'Content' => _t('Block.Content', 'Content'),
			)
		);
	}
}


1			<?php
2
3			class ContentBlock extends Block
			0 ignored issues – show Coding Style Compatibility introduced 2016-05-23 01:41 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4			{
5			/**
6			* If the singular name is set in a private static $singular_name, it cannot be changed using the translation files
7			* for some reason. Fix it by defining a method that handles the translation.
8			* @return string
9			*/
10			public function singular_name()
11			{
12			return _t('ContentBlock.SINGULARNAME', 'Content Block');
13			}
14
15			/**
16			* If the plural name is set in a private static $plural_name, it cannot be changed using the translation files
17			* for some reason. Fix it by defining a method that handles the translation.
18			* @return string
19			*/
20			public function plural_name()
21			{
22			return _t('ContentBlock.PLURALNAME', 'Content Blocks');
23			}
24
25			private static $db = array(
			0 ignored issues – show Comprehensibility introduced 2016-05-23 01:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-05-23 01:41 UTC by Report Bug Copy Issue Report The property `$db` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
26			'Content' => 'HTMLText',
27			);
28
29			public function fieldLabels($includeRelations = true)
30			{
31			return array_merge(
32			parent::fieldLabels($includeRelations),
33			array(
34			'Content' => _t('Block.Content', 'Content'),
35			)
36			);
37			}
38			}
39

sheadawson / silverstripe-blocks

GitHub Access Token became invalid

Push — master ( a4eef9...fb22a5 )

ContentBlock::fieldLabels() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like