@@ -328,8 +328,7 @@ public function installExtensionFromFolder($path, $type = 'Extension')
328328 $ I ->click (['link ' => 'Install from Folder ' ]);
329329 $ this ->debug ('I enter the Path ' );
330330 $ I ->fillField (['id ' => 'install_directory ' ], $ path );
331- // @todo: we need to find a better locator for the following Install button
332- $ I ->click (['xpath ' => "//button[contains(@onclick,'Joomla.submitbutton3()')] " ]); // Install button
331+ $ I ->click (['id ' => 'installbutton_directory ' ]); // Install button
333332 $ I ->waitForText ('was successful ' ,'60 ' , ['id ' => 'system-message-container ' ]);
334333 $ this ->debug ("$ type successfully installed from $ path " );
335334 }
@@ -349,9 +348,8 @@ public function installExtensionFromUrl($url, $type = 'Extension')
349348 $ I ->waitForText ('Extensions: Install ' ,'30 ' , ['css ' => 'H1 ' ]);
350349 $ I ->click (['link ' => 'Install from URL ' ]);
351350 $ this ->debug ('I enter the url ' );
352- $ I ->fillField (['id ' => 'install_url ' ], $ url );
353- // @todo: we need to find a better locator for the following Install button
354- $ I ->click (['xpath ' => "//button[contains(@onclick,'Joomla.submitbutton4()')] " ]); // Install button
351+ $ I ->fillField (['id ' => 'install_url ' ], $ url );
352+ $ I ->click (['id ' => 'installbutton_url ' ]); // Install button
355353 $ I ->waitForText ('was successful ' ,'30 ' , ['id ' => 'system-message-container ' ]);
356354 if ($ type == 'Extension ' )
357355 {
0 commit comments