Inspection of "SilverStripe 4 upgrade" - dynamic/silverstripe-product-catalog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#24)

by Jason

created 2017-07-12 21:02 UTC

Status

Doc Comments +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -171,7 +171,7 @@  discard block
 block discarded – undo
     }
 
     /**
-     * @return FieldList
+     * @return \SilverStripe\Forms\FieldList
      */
     public function getCMSFields()
     {
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
     /**
      * set ParentPage for ViewableDataobject
      *
-     * @return string
+     * @return CatalogCategory
      */
     public function getParentPage()
     {
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canEdit($member = null, $context = [])
     {
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canDelete($member = null, $context = [])
     {
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {

Please login to merge, or discard this patch.

code/docs/CareCleaningDoc.php 1 patch

Doc Comments +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -45,7 +45,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canEdit($member = null, $context = [])
     {
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canDelete($member = null, $context = [])
     {

Please login to merge, or discard this patch.

code/docs/OperationManual.php 1 patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -18,7 +18,7 @@  discard block
 block discarded – undo
     private static $plural_name = 'OperationManuals';
 
     /**
-     * @return FieldList
+     * @return \SilverStripe\Forms\FieldList
      */
     public function getCMSFields()
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canEdit($member = null, $context = [])
     {
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canDelete($member = null, $context = [])
     {

Please login to merge, or discard this patch.

code/docs/SpecSheet.php 1 patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -18,7 +18,7 @@  discard block
 block discarded – undo
     private static $plural_name = 'Spec Sheets';
 
     /**
-     * @return FieldList
+     * @return \SilverStripe\Forms\FieldList
      */
     public function getCMSFields()
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canEdit($member = null, $context = [])
     {
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canDelete($member = null, $context = [])
     {

Please login to merge, or discard this patch.

code/docs/Warranty.php 1 patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -18,7 +18,7 @@  discard block
 block discarded – undo
     private static $plural_name = 'Warranties';
 
     /**
-     * @return FieldList
+     * @return \SilverStripe\Forms\FieldList
      */
     public function getCMSFields()
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canEdit($member = null, $context = [])
     {
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
     /**
      * @param null $member
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public function canDelete($member = null, $context = [])
     {

Please login to merge, or discard this patch.

		@@ -171,7 +171,7 @@ discard block
		block discarded – undo
171	171	}
172	172
173	173	/**
174		- * @return FieldList
	174	+ * @return \SilverStripe\Forms\FieldList
175	175	*/
176	176	public function getCMSFields()
177	177	{
		@@ -276,7 +276,7 @@ discard block
		block discarded – undo
276	276	/**
277	277	* set ParentPage for ViewableDataobject
278	278	*
279		- * @return string
	279	+ * @return CatalogCategory
280	280	*/
281	281	public function getParentPage()
282	282	{
		@@ -316,7 +316,7 @@ discard block
		block discarded – undo
316	316	/**
317	317	* @param null $member
318	318	*
319		- * @return bool\|int
	319	+ * @return boolean\|string
320	320	*/
321	321	public function canEdit($member = null, $context = [])
322	322	{
		@@ -326,7 +326,7 @@ discard block
		block discarded – undo
326	326	/**
327	327	* @param null $member
328	328	*
329		- * @return bool\|int
	329	+ * @return boolean\|string
330	330	*/
331	331	public function canDelete($member = null, $context = [])
332	332	{
		@@ -336,7 +336,7 @@ discard block
		block discarded – undo
336	336	/**
337	337	* @param null $member
338	338	*
339		- * @return bool\|int
	339	+ * @return boolean\|string
340	340	*/
341	341	public function canCreate($member = null, $context = [])
342	342	{

		@@ -45,7 +45,7 @@ discard block
		block discarded – undo
45	45	/**
46	46	* @param null $member
47	47	*
48		- * @return bool\|int
	48	+ * @return boolean\|string
49	49	*/
50	50	public function canCreate($member = null, $context = [])
51	51	{
		@@ -55,7 +55,7 @@ discard block
		block discarded – undo
55	55	/**
56	56	* @param null $member
57	57	*
58		- * @return bool\|int
	58	+ * @return boolean\|string
59	59	*/
60	60	public function canEdit($member = null, $context = [])
61	61	{
		@@ -65,7 +65,7 @@ discard block
		block discarded – undo
65	65	/**
66	66	* @param null $member
67	67	*
68		- * @return bool\|int
	68	+ * @return boolean\|string
69	69	*/
70	70	public function canDelete($member = null, $context = [])
71	71	{

		@@ -18,7 +18,7 @@ discard block
		block discarded – undo
18	18	private static $plural_name = 'OperationManuals';
19	19
20	20	/**
21		- * @return FieldList
	21	+ * @return \SilverStripe\Forms\FieldList
22	22	*/
23	23	public function getCMSFields()
24	24	{
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	/**
45	45	* @param null $member
46	46	*
47		- * @return bool\|int
	47	+ * @return boolean\|string
48	48	*/
49	49	public function canCreate($member = null, $context = [])
50	50	{
		@@ -54,7 +54,7 @@ discard block
		block discarded – undo
54	54	/**
55	55	* @param null $member
56	56	*
57		- * @return bool\|int
	57	+ * @return boolean\|string
58	58	*/
59	59	public function canEdit($member = null, $context = [])
60	60	{
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	/**
65	65	* @param null $member
66	66	*
67		- * @return bool\|int
	67	+ * @return boolean\|string
68	68	*/
69	69	public function canDelete($member = null, $context = [])
70	70	{

		@@ -18,7 +18,7 @@ discard block
		block discarded – undo
18	18	private static $plural_name = 'Spec Sheets';
19	19
20	20	/**
21		- * @return FieldList
	21	+ * @return \SilverStripe\Forms\FieldList
22	22	*/
23	23	public function getCMSFields()
24	24	{
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	/**
45	45	* @param null $member
46	46	*
47		- * @return bool\|int
	47	+ * @return boolean\|string
48	48	*/
49	49	public function canCreate($member = null, $context = [])
50	50	{
		@@ -54,7 +54,7 @@ discard block
		block discarded – undo
54	54	/**
55	55	* @param null $member
56	56	*
57		- * @return bool\|int
	57	+ * @return boolean\|string
58	58	*/
59	59	public function canEdit($member = null, $context = [])
60	60	{
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	/**
65	65	* @param null $member
66	66	*
67		- * @return bool\|int
	67	+ * @return boolean\|string
68	68	*/
69	69	public function canDelete($member = null, $context = [])
70	70	{

		@@ -18,7 +18,7 @@ discard block
		block discarded – undo
18	18	private static $plural_name = 'Warranties';
19	19
20	20	/**
21		- * @return FieldList
	21	+ * @return \SilverStripe\Forms\FieldList
22	22	*/
23	23	public function getCMSFields()
24	24	{
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	/**
45	45	* @param null $member
46	46	*
47		- * @return bool\|int
	47	+ * @return boolean\|string
48	48	*/
49	49	public function canCreate($member = null, $context = [])
50	50	{
		@@ -54,7 +54,7 @@ discard block
		block discarded – undo
54	54	/**
55	55	* @param null $member
56	56	*
57		- * @return bool\|int
	57	+ * @return boolean\|string
58	58	*/
59	59	public function canEdit($member = null, $context = [])
60	60	{
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	/**
65	65	* @param null $member
66	66	*
67		- * @return bool\|int
	67	+ * @return boolean\|string
68	68	*/
69	69	public function canDelete($member = null, $context = [])
70	70	{

dynamic / silverstripe-product-catalog

Pull Request — master (#24)

Status

Category

Doc Comments +5 added lines, -5 removed lines patch added patch discarded remove patch

Doc Comments +3 added lines, -3 removed lines patch added patch discarded remove patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch

Doc Comments +4 added lines, -4 removed lines patch added patch discarded remove patch