See #3052703: Deprecate drupal_installation_attempted()

+++ b/core/lib/Drupal/Core/Config/ConfigInstaller.php
@@ -701,9 +702,17 @@ protected function drupalGetProfile() {
+   * @see https://www.drupal.org/node/3035275
...
+    @trigger_error(__METHOD__ . '() is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Use \Drupal\Core\Installer\InstallerKernel::installationAttempted() instead. See https://www.drupal.org/node/3035275', E_USER_DEPRECATED);

+++ b/core/lib/Drupal/Core/Form/FormSubmitter.php
@@ -147,9 +148,17 @@ public function redirectForm(FormStateInterface $form_state) {
+   * @see https://www.drupal.org/node/3035275

This is a pointed to the wrong change record. Should be https://www.drupal.org/node/3052704

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito created an issue. See original summary.

Gnanagowthaman sankar’s picture

Hi @penyaskito

Here by attached the patch. Please let me know for any changes.

Thanks & Regards,
Gnanagowthaman sankar

Gnanagowthaman sankar’s picture

Patch

Thanks & Regards,
Gnanagowthaman sankar

mandclu’s picture

Assigned: Unassigned » mandclu
Issue tags: +ContributionWeekend2020
mandclu’s picture

Status: Needs review » Reviewed & tested by the community

The patch applies as intended and successfully updates both the docblock and the link in the error message.

  • catch committed 31ff469 on 8.9.x
    Issue #3098521 by Gnanagowthaman sankar, mandclu, penyaskito:...

  • catch committed 70a86d7 on 9.0.x
    Issue #3098521 by Gnanagowthaman sankar, mandclu, penyaskito:...

  • catch committed a31620b on 8.8.x
    Issue #3098521 by Gnanagowthaman sankar, mandclu, penyaskito:...
catch’s picture

Version: 8.9.x-dev » 8.8.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.0.x/8.9.x/8.8.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.