EventsAdmin::toggleTicketIsTaxableForTicketRow() - Code Metrics - Inspection of "Daily Inspection: only show bulk action help tab w..." - eventespresso/event-espresso-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — BUG-10636-remove-unnecessary-b... (5637a2)

unknown

created 2017-06-29 21:18 UTC

EventsAdmin::toggleTicketIsTaxableForTicketRow() A

↳ Parent: EventsAdmin

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php
namespace EventEspresso\Codeception\helpers;

use Page\CoreAdmin;
use Page\EventsAdmin as EventsPage;


/**
 * Trait EventsAdmin
 * Helper actions for the Events Admin pages.
 *
 * @package EventEspresso\Codeception\helpers
 */
trait EventsAdmin
{

    /**
     * @param string $additional_params
     */
    public function amOnDefaultEventsListTablePage($additional_params = '')
    {
        $this->actor()->amOnAdminPage(EventsPage::defaultEventsListTableUrl($additional_params));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Triggers the publishing of the Event.
     */
    public function publishEvent()
    {
        $this->actor()->click(EventsPage::EVENT_EDITOR_PUBLISH_BUTTON_SELECTOR);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Triggers saving the Event.
     */
    public function saveEvent()
    {
        $this->actor()->click(EventsPage::EVENT_EDITOR_SAVE_BUTTON_SELECTOR);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Navigates the actor to the event list table page and will attempt to edit the event for the given title.
     * First this will search using the given title and then attempt to edit from the results of the search.
     *
     * Assumes actor is already logged in.
     * @param $event_title
     */
    public function amEditingTheEventWithTitle($event_title)
    {
        $this->amOnDefaultEventsListTablePage();
        $this->actor()->fillField(EventsPage::EVENT_LIST_TABLE_SEARCH_INPUT_SELECTOR, $event_title);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->actor()->click(CoreAdmin::LIST_TABLE_SEARCH_SUBMIT_SELECTOR);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->actor()->waitForText('Displaying search results for');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->actor()->click(EventsPage::eventListTableEventTitleEditLink($event_title));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Navigates the user to the single event page (frontend view) for the given event title via clicking the "View"
     * link for the event in the event list table.
     * Assumes the actor is already logged in and on the Event list table page.
     *
     * @param string $event_title
     */
    public function amOnEventPageAfterClickingViewLinkInListTableForEvent($event_title)
    {
        $this->actor()->moveMouseOver(EventsPage::eventListTableEventTitleEditLinkSelectorForTitle($event_title));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->actor()->click(EventsPage::eventListTableEventTitleViewLinkSelectorForTitle($event_title));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * This performs the click action on the gear icon that triggers the advanced settings view state.
     * Assumes the actor is already logged in and editing an event.
     *
     * @param int $row_number  What ticket row to toggle open/close.
     */
    public function toggleAdvancedSettingsViewForTicketRow($row_number = 1)
    {
        $this->actor()->click(EventsPage::eventEditorTicketAdvancedDetailsSelector($row_number));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Toggles the TKT_is_taxable checkbox for the ticket in the given row.
     * Assumes the actor is already logged in and editing an event and that the advanced settings view state for that
     * ticket is "open".
     *
     * @param int $row_number  What ticket row to toggle the checkbox for.
     */
    public function toggleTicketIsTaxableForTicketRow($row_number = 1)
    {
        $this->actor()->click(EventsPage::eventEditorTicketTaxableCheckboxSelector($row_number));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }


    /**
     * Use to change the default registration status for the event.
     * Assumes the view is already on the event editor.
     * @param $registration_status
     */
    public function changeDefaultRegistrationStatusTo($registration_status)
    {
        $this->actor()->selectOption(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            EventsPage::EVENT_EDITOR_DEFAULT_REGISTRATION_STATUS_FIELD_SELECTOR,
            $registration_status
        );
    }


    /**
     * Use this from the context of the event editor to select the given custom template for a given message type and
     * messenger.
     *
     * @param string $message_type_label  The visible label for the message type (eg Registration Approved)
     * @param string $messenger_slug      The slug for the messenger (eg 'email')
     * @param string $custom_template_label The visible label in the select input for the custom template you want
     *                                      selected.
     */
    public function selectCustomTemplateFor($message_type_label, $messenger_slug, $custom_template_label)
    {
        $this->actor()->click(EventsPage::eventEditorNotificationsMetaBoxMessengerTabSelector($messenger_slug));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->actor()->selectOption(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            EventsPage::eventEditorNotificationsMetaBoxSelectSelectorForMessageType($message_type_label),
            $custom_template_label
        );
    }
}

1			<?php
2			namespace EventEspresso\Codeception\helpers;
3
4			use Page\CoreAdmin;
5			use Page\EventsAdmin as EventsPage;
6
7
8			/**
9			* Trait EventsAdmin
10			* Helper actions for the Events Admin pages.
11			*
12			* @package EventEspresso\Codeception\helpers
13			*/
14			trait EventsAdmin
15			{
16
17			/**
18			* @param string $additional_params
19			*/
20			public function amOnDefaultEventsListTablePage($additional_params = '')
21			{
22			$this->actor()->amOnAdminPage(EventsPage::defaultEventsListTableUrl($additional_params));
			0 ignored issues – show Bug introduced 2017-05-19 23:04 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
23			}
24
25
26			/**
27			* Triggers the publishing of the Event.
28			*/
29			public function publishEvent()
30			{
31			$this->actor()->click(EventsPage::EVENT_EDITOR_PUBLISH_BUTTON_SELECTOR);
			0 ignored issues – show Bug introduced 2017-05-19 23:04 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
32			}
33
34
35			/**
36			* Triggers saving the Event.
37			*/
38			public function saveEvent()
39			{
40			$this->actor()->click(EventsPage::EVENT_EDITOR_SAVE_BUTTON_SELECTOR);
			0 ignored issues – show Bug introduced 2017-06-28 22:17 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
41			}
42
43
44			/**
45			* Navigates the actor to the event list table page and will attempt to edit the event for the given title.
46			* First this will search using the given title and then attempt to edit from the results of the search.
47			*
48			* Assumes actor is already logged in.
49			* @param $event_title
50			*/
51			public function amEditingTheEventWithTitle($event_title)
52			{
53			$this->amOnDefaultEventsListTablePage();
54			$this->actor()->fillField(EventsPage::EVENT_LIST_TABLE_SEARCH_INPUT_SELECTOR, $event_title);
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
55			$this->actor()->click(CoreAdmin::LIST_TABLE_SEARCH_SUBMIT_SELECTOR);
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
56			$this->actor()->waitForText('Displaying search results for');
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
57			$this->actor()->click(EventsPage::eventListTableEventTitleEditLink($event_title));
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
58			}
59
60
61			/**
62			* Navigates the user to the single event page (frontend view) for the given event title via clicking the "View"
63			* link for the event in the event list table.
64			* Assumes the actor is already logged in and on the Event list table page.
65			*
66			* @param string $event_title
67			*/
68			public function amOnEventPageAfterClickingViewLinkInListTableForEvent($event_title)
69			{
70			$this->actor()->moveMouseOver(EventsPage::eventListTableEventTitleEditLinkSelectorForTitle($event_title));
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
71			$this->actor()->click(EventsPage::eventListTableEventTitleViewLinkSelectorForTitle($event_title));
			0 ignored issues – show Bug introduced 2017-05-19 23:16 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
72			}
73
74
75			/**
76			* This performs the click action on the gear icon that triggers the advanced settings view state.
77			* Assumes the actor is already logged in and editing an event.
78			*
79			* @param int $row_number What ticket row to toggle open/close.
80			*/
81			public function toggleAdvancedSettingsViewForTicketRow($row_number = 1)
82			{
83			$this->actor()->click(EventsPage::eventEditorTicketAdvancedDetailsSelector($row_number));
			0 ignored issues – show Bug introduced 2017-06-21 22:35 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
84			}
85
86
87			/**
88			* Toggles the TKT_is_taxable checkbox for the ticket in the given row.
89			* Assumes the actor is already logged in and editing an event and that the advanced settings view state for that
90			* ticket is "open".
91			*
92			* @param int $row_number What ticket row to toggle the checkbox for.
93			*/
94			public function toggleTicketIsTaxableForTicketRow($row_number = 1)
95			{
96			$this->actor()->click(EventsPage::eventEditorTicketTaxableCheckboxSelector($row_number));
			0 ignored issues – show Bug introduced 2017-06-21 22:35 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
97			}
98
99
100			/**
101			* Use to change the default registration status for the event.
102			* Assumes the view is already on the event editor.
103			* @param $registration_status
104			*/
105			public function changeDefaultRegistrationStatusTo($registration_status)
106			{
107			$this->actor()->selectOption(
			0 ignored issues – show Bug introduced 2017-05-25 21:58 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
108			EventsPage::EVENT_EDITOR_DEFAULT_REGISTRATION_STATUS_FIELD_SELECTOR,
109			$registration_status
110			);
111			}
112
113
114			/**
115			* Use this from the context of the event editor to select the given custom template for a given message type and
116			* messenger.
117			*
118			* @param string $message_type_label The visible label for the message type (eg Registration Approved)
119			* @param string $messenger_slug The slug for the messenger (eg 'email')
120			* @param string $custom_template_label The visible label in the select input for the custom template you want
121			* selected.
122			*/
123			public function selectCustomTemplateFor($message_type_label, $messenger_slug, $custom_template_label)
124			{
125			$this->actor()->click(EventsPage::eventEditorNotificationsMetaBoxMessengerTabSelector($messenger_slug));
			0 ignored issues – show Bug introduced 2017-06-09 21:33 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
126			$this->actor()->selectOption(
			0 ignored issues – show Bug introduced 2017-06-09 21:33 UTC by Report Bug Copy Issue Report It seems like `actor()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
127			EventsPage::eventEditorNotificationsMetaBoxSelectSelectorForMessageType($message_type_label),
128			$custom_template_label
129			);
130			}
131			}

eventespresso / event-espresso-core

Branch — BUG-10636-remove-unnecessary-b... (5637a2)

EventsAdmin::toggleTicketIsTaxableForTicketRow() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like