comparison core/modules/big_pipe/src/Render/BigPipe.php @ 17:129ea1e6d783

Update, including to Drupal core 8.6.10
author Chris Cannam
date Thu, 28 Feb 2019 13:21:36 +0000
parents 1fec387a4317
children
comparison
equal deleted inserted replaced
16:c2387f117808 17:129ea1e6d783
728 // The 'status messages' placeholder needs to be special cased, because it 728 // The 'status messages' placeholder needs to be special cased, because it
729 // depends on global state that can be modified when other placeholders are 729 // depends on global state that can be modified when other placeholders are
730 // being rendered: any code can add messages to render. 730 // being rendered: any code can add messages to render.
731 // This violates the principle that each lazy builder must be able to render 731 // This violates the principle that each lazy builder must be able to render
732 // itself in isolation, and therefore in any order. However, we cannot 732 // itself in isolation, and therefore in any order. However, we cannot
733 // change the way drupal_set_message() works in the Drupal 8 cycle. So we 733 // change the way \Drupal\Core\Messenger\MessengerInterface::addMessage()
734 // have to accommodate its special needs. 734 // works in the Drupal 8 cycle. So we have to accommodate its special needs.
735 // Allowing placeholders to be rendered in a particular order (in this case: 735 // Allowing placeholders to be rendered in a particular order (in this case:
736 // last) would violate this isolation principle. Thus a monopoly is granted 736 // last) would violate this isolation principle. Thus a monopoly is granted
737 // to this one special case, with this hard-coded solution. 737 // to this one special case, with this hard-coded solution.
738 // @see \Drupal\Core\Render\Element\StatusMessages 738 // @see \Drupal\Core\Render\Element\StatusMessages
739 // @see \Drupal\Core\Render\Renderer::replacePlaceholders() 739 // @see \Drupal\Core\Render\Renderer::replacePlaceholders()