TrainingPage_Controller::SignUpForm() - Code Metrics - sunnysideup/silverstripe-traininglistingsandsignup - Measure and Improve Code Quality continuously with Scrutinizer

TrainingPage_Controller::SignUpForm() B
last analyzed 2018-03-20 01:36 UTC

↳ Parent: TrainingPage_Controller

Complexity

Conditions	5
Paths	2

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	8.8571
c	0
b	0
f	0
cc	5
eloc	9
nc	2
nop	0

<?php

class TrainingPage extends Page
namespace YourVendor;

class YourClass { }
{
    private static $icon = "mysite/images/treeicons/TrainingPage";


    private static $db = array(

        "Date" => "Date",
        "EndDate" => "Date",
        "Location" => "Varchar(255)",
        "Price" => "Currency",
        "IsOpenForBookings" => "Boolean",
        "PlacesAvailable" => "Int",
        "PeopleSignedUpElseWhere" => "Int",
        "MoreInformation" => "HTMLText",
        "Options" => "Text"
    );

    private static $has_one = array(

        "DownloadFile" => "File"
    );

    private static $many_many = array(

        "Attendees" => "Member"
    );

    private static $many_many_extraFields = array(

         "Attendees" => array(
                "SelectedOption" => "Varchar(255)",
                "BookingCode" => "Varchar(255)"
         )
    );

    //parents and children in sitetree
    private static $allowed_children = "none"; //can also be "none";

    private static $default_parent = "TrainingHolder";

    private static $can_be_root = false; //default is true


    public function getCMSFields()

    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab("Root.WhoWhereWhat", new DateField("Date", "Start Date"));
        $fields->addFieldToTab("Root.WhoWhereWhat", new DateField("EndDate", "End Date - can be left blank for one day events"));
        $fields->addFieldToTab("Root.WhoWhereWhat", new TextField("Location"));
        $fields->addFieldToTab("Root.WhoWhereWhat", new CurrencyField("Price"));
        $fields->addFieldToTab("Root.MoreInformation", new UploadField("DownloadFile", "Download File"));
        $fields->addFieldToTab("Root.MoreInformation", new HtmlEditorField("MoreInformation", "More Information"));
        $fields->addFieldToTab("Root.Bookings", new CheckboxField("IsOpenForBookings", "Is Open For Bookings"));
        $fields->addFieldToTab("Root.Bookings", new HeaderField("ActualPlacesAvailableHeader", "Actual Places Available: ".$this->ActualPlacesAvailable(), 3));
        $fields->addFieldToTab("Root.Bookings", new LiteralField("ActualPlacesAvailableData", "Calculated as: Places Available [-] Minus People Signed up elsewhere [-] Minus People Signed up through this Website)"));
        $fields->addFieldToTab("Root.Bookings", new NumericField("PlacesAvailable", "Places Available"));
        $fields->addFieldToTab("Root.Bookings", new NumericField("PeopleSignedUpElseWhere", "People Signed Up Else Where (thus excluding the ones signed up on this website)"));
        $fields->addFieldToTab("Root.Bookings", new HeaderField("FormAdditions", "Form Additions", 3));
        $fields->addFieldToTab("Root.Bookings", new TextareaField("Options", "Options available (separate by comma) - if any (e.g. venues)"));
        $fields->addFieldToTab("Root.Bookings", new HeaderField("Current Registrations", "Current Registrations", 3));
        $fields->addFieldToTab(
            "Root.Bookings",
            $this->MemberField()
        );
        return $fields;
    }

    public function MemberField()
    {
        $memberField = new GridField(
            $name = "Attendees",
            $sourceClass = "Attendees",
            $this->Attendees(),
            GridFieldConfig_RelationEditor::create()
        );
        return $memberField;
    }

    public function addAttendee($member, $extraFields = null)
    {
        $existingMembers = $this->Attendees();
        $existingMembers->add($member, $extraFields);
    }

    public function DifferentEndDate()
    {
        if ($this->Date != $this->EndDate && $this->EndDate) {
            return true;
        }
    }

    public function DifferentEndMonth()
    {
        if ($this->DifferentEndDate()) {
            if (Date("F", $this->Date) || Date("F", $this->EndDate)) {
                return true;
            }
        }
    }

    public function ActualPlacesAvailable()
    {
        return intval($this->PlacesAvailable - $this->PeopleSignedUpElseWhere - $this->Attendees("\"TrainingPageID\" = ".$this->ID)->count());
    }
}

class TrainingPage_Controller extends Page_Controller
namespace YourVendor;

class YourClass { }
{
    private static $allowed_actions = array(

        "thankyou",
        "SignUpForm"
    );

    public function SignUpForm()
    {
        if (
            !$this->IsOpenForBookings ||
            "thankyou" == $this->getRequest()->param("Action") ||
            $this->MemberAlreadySignedUp() ||
            $this->ActualPlacesAvailable() < 1
        ) {
            return false;
        }
        $form = new TrainingSignupForm($this, "SignUpForm", "Sign-Up for ".$this->Title);
        return $form;
    }

    public function thankyou()
    {
        $this->Title = "Thank You";
        $this->Content = "We will be in touch soon";
        return array();
    }

    public function MemberAlreadySignedUp()
    {
        if ($id = Member::currentUserID()) {
            if ($this->Attendees("\"MemberID\" = ".$id.' AND \"TrainingPageID\" = '.$this->ID)->count()) {
                return true;
            }
        }
        return false;
    }
}


1			<?php
2
3			class TrainingPage extends Page
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:37 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			private static $icon = "mysite/images/treeicons/TrainingPage";
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$icon` 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...
6
7			private static $db = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 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...
8			"Date" => "Date",
9			"EndDate" => "Date",
10			"Location" => "Varchar(255)",
11			"Price" => "Currency",
12			"IsOpenForBookings" => "Boolean",
13			"PlacesAvailable" => "Int",
14			"PeopleSignedUpElseWhere" => "Int",
15			"MoreInformation" => "HTMLText",
16			"Options" => "Text"
17			);
18
19			private static $has_one = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$has_one` 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...
20			"DownloadFile" => "File"
21			);
22
23			private static $many_many = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$many_many` 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...
24			"Attendees" => "Member"
25			);
26
27			private static $many_many_extraFields = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$many_many_extraFields` 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...
28			"Attendees" => array(
29			"SelectedOption" => "Varchar(255)",
30			"BookingCode" => "Varchar(255)"
31			)
32			);
33
34			//parents and children in sitetree
35			private static $allowed_children = "none"; //can also be "none";
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$allowed_children` 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...
36			private static $default_parent = "TrainingHolder";
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$default_parent` 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...
37			private static $can_be_root = false; //default is true
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$can_be_root` 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...
38
39			public function getCMSFields()
			0 ignored issues – show Documentation introduced 2018-02-21 04:23 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
40			{
41			$fields = parent::getCMSFields();
42			$fields->addFieldToTab("Root.WhoWhereWhat", new DateField("Date", "Start Date"));
43			$fields->addFieldToTab("Root.WhoWhereWhat", new DateField("EndDate", "End Date - can be left blank for one day events"));
44			$fields->addFieldToTab("Root.WhoWhereWhat", new TextField("Location"));
45			$fields->addFieldToTab("Root.WhoWhereWhat", new CurrencyField("Price"));
46			$fields->addFieldToTab("Root.MoreInformation", new UploadField("DownloadFile", "Download File"));
47			$fields->addFieldToTab("Root.MoreInformation", new HtmlEditorField("MoreInformation", "More Information"));
48			$fields->addFieldToTab("Root.Bookings", new CheckboxField("IsOpenForBookings", "Is Open For Bookings"));
49			$fields->addFieldToTab("Root.Bookings", new HeaderField("ActualPlacesAvailableHeader", "Actual Places Available: ".$this->ActualPlacesAvailable(), 3));
50			$fields->addFieldToTab("Root.Bookings", new LiteralField("ActualPlacesAvailableData", "Calculated as: Places Available [-] Minus People Signed up elsewhere [-] Minus People Signed up through this Website)"));
51			$fields->addFieldToTab("Root.Bookings", new NumericField("PlacesAvailable", "Places Available"));
52			$fields->addFieldToTab("Root.Bookings", new NumericField("PeopleSignedUpElseWhere", "People Signed Up Else Where (thus excluding the ones signed up on this website)"));
53			$fields->addFieldToTab("Root.Bookings", new HeaderField("FormAdditions", "Form Additions", 3));
54			$fields->addFieldToTab("Root.Bookings", new TextareaField("Options", "Options available (separate by comma) - if any (e.g. venues)"));
55			$fields->addFieldToTab("Root.Bookings", new HeaderField("Current Registrations", "Current Registrations", 3));
56			$fields->addFieldToTab(
57			"Root.Bookings",
58			$this->MemberField()
59			);
60			return $fields;
61			}
62
63			public function MemberField()
64			{
65			$memberField = new GridField(
66			$name = "Attendees",
67			$sourceClass = "Attendees",
68			$this->Attendees(),
69			GridFieldConfig_RelationEditor::create()
70			);
71			return $memberField;
72			}
73
74			public function addAttendee($member, $extraFields = null)
75			{
76			$existingMembers = $this->Attendees();
77			$existingMembers->add($member, $extraFields);
78			}
79
80			public function DifferentEndDate()
81			{
82			if ($this->Date != $this->EndDate && $this->EndDate) {
83			return true;
84			}
85			}
86
87			public function DifferentEndMonth()
88			{
89			if ($this->DifferentEndDate()) {
90			if (Date("F", $this->Date) \|\| Date("F", $this->EndDate)) {
91			return true;
92			}
93			}
94			}
95
96			public function ActualPlacesAvailable()
97			{
98			return intval($this->PlacesAvailable - $this->PeopleSignedUpElseWhere - $this->Attendees("\"TrainingPageID\" = ".$this->ID)->count());
99			}
100			}
101
102			class TrainingPage_Controller extends Page_Controller
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2017-01-03 01:37 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...
103			{
104			private static $allowed_actions = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:37 UTC by Report Bug Copy Issue Report The property `$allowed_actions` 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...
105			"thankyou",
106			"SignUpForm"
107			);
108
109			public function SignUpForm()
110			{
111			if (
112			!$this->IsOpenForBookings \|\|
113			"thankyou" == $this->getRequest()->param("Action") \|\|
114			$this->MemberAlreadySignedUp() \|\|
115			$this->ActualPlacesAvailable() < 1
116			) {
117			return false;
118			}
119			$form = new TrainingSignupForm($this, "SignUpForm", "Sign-Up for ".$this->Title);
120			return $form;
121			}
122
123			public function thankyou()
124			{
125			$this->Title = "Thank You";
126			$this->Content = "We will be in touch soon";
127			return array();
128			}
129
130			public function MemberAlreadySignedUp()
131			{
132			if ($id = Member::currentUserID()) {
133			if ($this->Attendees("\"MemberID\" = ".$id.' AND \"TrainingPageID\" = '.$this->ID)->count()) {
134			return true;
135			}
136			}
137			return false;
138			}
139			}
140

sunnysideup / silverstripe-traininglistingsandsignup

TrainingPage_Controller::SignUpForm() B last analyzed 2018-03-20 01:36 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

TrainingPage_Controller::SignUpForm() B
last analyzed 2018-03-20 01:36 UTC