AkismetProcessor::preRequest() - Code Metrics - Inspection of "Merge pull request #12 from helpfulrobot/convert-t..." - silverstripe/silverstripe-akismet - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1d7363...6e2aa4 )

by Daniel

created 2015-11-25 15:36 UTC

AkismetProcessor::preRequest() A

↳ Parent: AkismetProcessor

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	18
rs	9.2
cc	4
eloc	8
nc	4
nop	3

<?php

/**
 * Allows akismet to be configured via siteconfig instead of hard-coded configuration
 */
class AkismetProcessor implements RequestFilter
namespace YourVendor;

class YourClass { }
{
    public function postRequest(SS_HTTPRequest $request, SS_HTTPResponse $response, DataModel $model)
    {
    }

    public function preRequest(SS_HTTPRequest $request, Session $session, DataModel $model)
    {
        // Skip if database isn't ready
        if (!$this->isDBReady()) {
            return;
        }

        // Skip if SiteConfig doesn't have this extension
        if (!SiteConfig::has_extension('AkismetConfig')) {
            return;
        }

        // Check if key exists
        $akismetKey = SiteConfig::current_site_config()->AkismetKey;
        if ($akismetKey) {
            AkismetSpamProtector::set_api_key($akismetKey);
        }
    }

    /**
     * Make sure the DB is ready before accessing siteconfig db field
     *
     * @return bool
     */
    protected function isDBReady()
    {
        if (!DB::isActive()) {

            return false;
        }

        // Require table
        if (!DB::getConn()->hasTable('SiteConfig')) {

            return false;
        }

        // Ensure siteconfig has all fields necessary
        $dbFields = DB::fieldList('SiteConfig');

        if (empty($dbFields)) {
            return false;
        }

        // Ensure that SiteConfig has all fields
        $objFields = DataObject::database_fields('SiteConfig', false);
        $missingFields = array_diff_key($objFields, $dbFields);
        return empty($missingFields);
    }
}


1			<?php
2
3			/**
4			* Allows akismet to be configured via siteconfig instead of hard-coded configuration
5			*/
6			class AkismetProcessor implements RequestFilter
			0 ignored issues – show Coding Style Compatibility introduced 2015-11-20 23:49 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...
7			{
8			public function postRequest(SS_HTTPRequest $request, SS_HTTPResponse $response, DataModel $model)
9			{
10			}
11
12			public function preRequest(SS_HTTPRequest $request, Session $session, DataModel $model)
13			{
14			// Skip if database isn't ready
15			if (!$this->isDBReady()) {
16			return;
17			}
18
19			// Skip if SiteConfig doesn't have this extension
20			if (!SiteConfig::has_extension('AkismetConfig')) {
21			return;
22			}
23
24			// Check if key exists
25			$akismetKey = SiteConfig::current_site_config()->AkismetKey;
26			if ($akismetKey) {
27			AkismetSpamProtector::set_api_key($akismetKey);
28			}
29			}
30
31			/**
32			* Make sure the DB is ready before accessing siteconfig db field
33			*
34			* @return bool
35			*/
36			protected function isDBReady()
37			{
38			if (!DB::isActive()) {
			0 ignored issues – show Deprecated Code introduced 2015-11-20 23:49 UTC by Report Bug Copy Issue Report The method `DB::isActive()` has been deprecated with message: since version 4.0 Use DB::is_active instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
39			return false;
40			}
41
42			// Require table
43			if (!DB::getConn()->hasTable('SiteConfig')) {
			0 ignored issues – show Deprecated Code introduced 2015-11-20 23:49 UTC by Report Bug Copy Issue Report The method `DB::getConn()` has been deprecated with message: since version 4.0 Use DB::get_conn instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history... Deprecated Code introduced 2015-11-20 23:49 UTC by Report Bug Copy Issue Report The method `SS_Database::hasTable()` has been deprecated with message: since version 4.0 Use DB::get_schema()->hasTable() instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
44			return false;
45			}
46
47			// Ensure siteconfig has all fields necessary
48			$dbFields = DB::fieldList('SiteConfig');
			0 ignored issues – show Deprecated Code introduced 2015-11-20 23:49 UTC by Report Bug Copy Issue Report The method `DB::fieldList()` has been deprecated with message: since version 4.0 Use DB::field_list instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
49			if (empty($dbFields)) {
50			return false;
51			}
52
53			// Ensure that SiteConfig has all fields
54			$objFields = DataObject::database_fields('SiteConfig', false);
55			$missingFields = array_diff_key($objFields, $dbFields);
56			return empty($missingFields);
57			}
58			}
59

silverstripe / silverstripe-akismet

Push — master ( 1d7363...6e2aa4 )

AkismetProcessor::preRequest() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like