Skip to content

Commit b39d5b2

Browse files
astridxyvesh
authored andcommitted
coding standards (#144)
1 parent 1bbb7b9 commit b39d5b2

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

src/JoomlaBrowser.php

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@ protected function instantiateLocator()
107107
* @param string|null $password Optional password. If not passed the one in acceptance.suite.yml will be used
108108
*
109109
* @return void
110+
*
110111
* @since 3.0.0
111112
*/
112113
public function doAdministratorLogin($user = null, $password = null)
@@ -143,6 +144,7 @@ public function doAdministratorLogin($user = null, $password = null)
143144
* @param string|null $password Optional password. If not passed the one in acceptance.suite.yml will be used
144145
*
145146
* @return void
147+
*
146148
* @since 3.0.0
147149
*/
148150
public function doFrontEndLogin($user = null, $password = null)
@@ -195,6 +197,7 @@ public function doFrontendLogout()
195197
* Installs Joomla
196198
*
197199
* @return void
200+
*
198201
* @since 3.0.0
199202
*/
200203
public function installJoomla()
@@ -284,6 +287,7 @@ public function installJoomla()
284287
* Install Joomla removing the Installation folder at the end of the execution
285288
*
286289
* @return void
290+
*
287291
* @since 3.0.0
288292
*/
289293
public function installJoomlaRemovingInstallationFolder()
@@ -308,6 +312,7 @@ public function installJoomlaRemovingInstallationFolder()
308312
* @example: $this->installJoomlaMultilingualSite(['Spanish', 'French']);
309313
*
310314
* @return void
315+
*
311316
* @since 3.0.0
312317
*/
313318
public function installJoomlaMultilingualSite($languages = array())
@@ -408,7 +413,6 @@ public function installExtensionFromDirectory($path, $type = 'Extension')
408413
*/
409414
public function installExtensionFromFolder($path, $type = 'Extension')
410415
{
411-
412416
$this->amOnPage('/administrator/index.php?option=com_installer');
413417
$this->waitForText('Extensions: Install', '30', ['css' => 'H1']);
414418
$this->click(['link' => 'Install from Folder']);
@@ -460,8 +464,8 @@ public function installExtensionFromUrl($url, $type = 'Extension')
460464
/**
461465
* Installs a Extension in Joomla using the file upload option
462466
*
463-
* @param string $file Path to the file in the _data folder
464-
* @param string $type Type of Extension
467+
* @param string $file Path to the file in the _data folder
468+
* @param string $type Type of Extension
465469
*
466470
* {@internal doAdminLogin() before}
467471
*
@@ -480,14 +484,17 @@ public function installExtensionFromFileUpload($file, $type = 'Extension')
480484
$this->attachFile(array('id' => 'install_package'), $file);
481485

482486
$this->waitForText('was successful', '30', array('id' => 'system-message-container'));
487+
483488
if ($type == 'Extension')
484489
{
485490
$this->debug('Extension successfully installed.');
486491
}
492+
487493
if ($type == 'Plugin')
488494
{
489495
$this->debug('Installing plugin was successful.');
490496
}
497+
491498
if ($type == 'Package')
492499
{
493500
$this->debug('Installation of the package was successful.');
@@ -533,7 +540,6 @@ public function checkForPhpNoticesOrWarnings($page = null)
533540
*/
534541
public function selectOptionInRadioField($label, $option)
535542
{
536-
537543
$this->debug("Trying to select the $option from the $label");
538544
$label = $this->findField(['xpath' => "//label[contains(normalize-space(string(.)), '$label')]"]);
539545
$radioId = $label->getAttribute('for');
@@ -650,7 +656,6 @@ public function selectMultipleOptionsInChosen($label, $options)
650656
$selectID = $select->getAttribute('id');
651657
$chosenSelectID = $selectID . '_chzn';
652658

653-
654659
foreach ($options as $option)
655660
{
656661
$this->debug("I open the $label chosen selector");
@@ -798,7 +803,6 @@ public function checkExistenceOf($name)
798803
*/
799804
public function checkAllResults()
800805
{
801-
802806
$this->debug("Selecting Checkall button");
803807
$this->click(['xpath' => "//thead//input[@name='checkall-toggle' or @name='toggle']"]);
804808
}
@@ -812,7 +816,6 @@ public function checkAllResults()
812816
*/
813817
public function installLanguage($languageName)
814818
{
815-
816819
$this->amOnPage('administrator/index.php?option=com_installer&view=languages');
817820
$this->debug('I check for Notices and Warnings');
818821
$this->checkForPhpNoticesOrWarnings();
@@ -869,6 +872,7 @@ public function setModulePosition($module, $position = 'position-7')
869872
* @param string $module The full name of the module
870873
*
871874
* @return void
875+
*
872876
* @since 3.0.0
873877
*/
874878
public function publishModule($module)
@@ -886,6 +890,7 @@ public function publishModule($module)
886890
* @param string $module The full name of the module
887891
*
888892
* @return void
893+
*
889894
* @since 3.0.0
890895
*/
891896
public function displayModuleOnAllPages($module)
@@ -908,6 +913,7 @@ public function displayModuleOnAllPages($module)
908913
* @param string $button The full name of the button
909914
*
910915
* @return void
916+
*
911917
* @since 3.0.0
912918
*/
913919
public function clickToolbarButton($button)

0 commit comments

Comments
 (0)