Unverified Commit 6a0d497e authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3180382 by phenaproxima, dww, tedbow: UpdateManagerUpdate doesn't need... 

Issue #3180382 by phenaproxima, dww, tedbow: UpdateManagerUpdate doesn't need to use the renderer service
parent 3a5c1135

core/modules/update/src/Form/UpdateManagerUpdate.php

+1 −4
Changes for core/modules/update/src/Form/UpdateManagerUpdate.php: 1 added line, 4 removed lines.
Original line number Diff line number Diff line
@@ -72,13 +72,10 @@ public static function create(ContainerInterface $container) { 
  public function buildForm(array $form, FormStateInterface $form_state) {

    $this->moduleHandler->loadInclude('update', 'inc', 'update.manager');



    $last_markup = [

    $form['last_check'] = [

      '#theme' => 'update_last_check',

      '#last' => $this->state->get('update.last_check', 0),

    ];

    $form['last_check'] = [

      '#markup' => \Drupal::service('renderer')->render($last_markup),

    ];



    if (!_update_manager_check_backends($form, 'update')) {

      return $form;

core/modules/update/tests/src/Functional/UpdateTestBase.php

+1 −0
Changes for core/modules/update/tests/src/Functional/UpdateTestBase.php: 1 added line, 0 removed lines.
Original line number Diff line number Diff line
@@ -110,6 +110,7 @@ protected function standardTests() { 
    // Verify that the link to the Drupal project appears.

    $this->assertRaw(Link::fromTextAndUrl(t('Drupal'), Url::fromUri('http://example.com/project/drupal'))->toString());

    $this->assertNoText('No available releases found');

    $this->assertSession()->pageTextContains('Last checked:');

  }



  /**