@backtrace', $error); + } + } + + $content = $this->t('The website encountered an unexpected error. Please try again later.'); + $content .= $message ? '' . $message : ''; + $response = new Response($content, 500); + + if ($exception instanceof HttpExceptionInterface) { + $response->setStatusCode($exception->getStatusCode()); + $response->headers->add($exception->getHeaders()); + } + else { + $response->setStatusCode(Response::HTTP_INTERNAL_SERVER_ERROR, '500 Service unavailable (with message)'); + } + + $event->setResponse($response); + } + + /** + * Handles any exception as a generic error page for JSON. + * + * @todo This should probably check the error reporting level. + * + * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event + * The event to process. + */ + protected function onJson(GetResponseForExceptionEvent $event) { + $exception = $event->getException(); + $error = Error::decodeException($exception); + + // Display the message if the current error reporting level allows this type + // of message to be displayed, + $data = NULL; + if (error_displayable($error) && $message = $exception->getMessage()) { + $data = ['message' => sprintf('A fatal error occurred: %s', $message)]; + } + + $response = new JsonResponse($data, Response::HTTP_INTERNAL_SERVER_ERROR); + if ($exception instanceof HttpExceptionInterface) { + $response->setStatusCode($exception->getStatusCode()); + $response->headers->add($exception->getHeaders()); + } + + $event->setResponse($response); + } + + /** + * Handles an HttpExceptionInterface exception for unknown formats. + * + * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event + * The event to process. + */ + protected function onFormatUnknown(GetResponseForExceptionEvent $event) { + /** @var \Symfony\Component\HttpKernel\Exception\HttpExceptionInterface|\Exception $exception */ + $exception = $event->getException(); + + $response = new Response($exception->getMessage(), $exception->getStatusCode(), $exception->getHeaders()); + $event->setResponse($response); + } + + /** + * Handles errors for this subscriber. + * + * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event + * The event to process. + */ + public function onException(GetResponseForExceptionEvent $event) { + $format = $this->getFormat($event->getRequest()); + $exception = $event->getException(); + + $method = 'on' . $format; + if (!method_exists($this, $method)) { + if ($exception instanceof HttpExceptionInterface) { + $this->onFormatUnknown($event); + $response = $event->getResponse(); + $response->headers->set('Content-Type', 'text/plain'); + } + else { + $this->onHtml($event); + } + } + else { + $this->$method($event); + } + } + + /** + * Gets the error-relevant format from the request. + * + * @param \Symfony\Component\HttpFoundation\Request $request + * The request object. + * + * @return string + * The format as which to treat the exception. + */ + protected function getFormat(Request $request) { + $format = $request->query->get(MainContentViewSubscriber::WRAPPER_FORMAT, $request->getRequestFormat()); + + // These are all JSON errors for our purposes. Any special handling for + // them can/should happen in earlier listeners if desired. + if (in_array($format, ['drupal_modal', 'drupal_dialog', 'drupal_ajax'])) { + $format = 'json'; + } + + // Make an educated guess that any Accept header type that includes "json" + // can probably handle a generic JSON response for errors. As above, for + // any format this doesn't catch or that wants custom handling should + // register its own exception listener. + foreach ($request->getAcceptableContentTypes() as $mime) { + if (strpos($mime, 'html') === FALSE && strpos($mime, 'json') !== FALSE) { + $format = 'json'; + } + } + + return $format; + } + + /** + * Registers the methods in this class that should be listeners. + * + * @return array + * An array of event listener definitions. + */ + public static function getSubscribedEvents() { + $events[KernelEvents::EXCEPTION][] = ['onException', -256]; + return $events; + } + +} diff --git a/core/lib/Drupal/Core/EventSubscriber/ExceptionJsonSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/ExceptionJsonSubscriber.php index 6ccb743..76cc55e 100644 --- a/core/lib/Drupal/Core/EventSubscriber/ExceptionJsonSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/ExceptionJsonSubscriber.php @@ -14,7 +14,7 @@ class ExceptionJsonSubscriber extends HttpExceptionSubscriberBase { * {@inheritdoc} */ protected function getHandledFormats() { - return ['json', 'drupal_modal', 'drupal_dialog', 'drupal_ajax']; + return ['json']; } /** diff --git a/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php deleted file mode 100644 index 4317ff0..0000000 --- a/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php +++ /dev/null @@ -1,195 +0,0 @@ -configFactory = $config_factory; - } - - /** - * Gets the configured error level. - * - * @return string - */ - protected function getErrorLevel() { - if (!isset($this->errorLevel)) { - $this->errorLevel = $this->configFactory->get('system.logging')->get('error_level'); - } - return $this->errorLevel; - } - - /** - * Handles exceptions for this subscriber. - * - * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event - * The event to process. - */ - public function onException(GetResponseForExceptionEvent $event) { - $exception = $event->getException(); - $error = Error::decodeException($exception); - - // Display the message if the current error reporting level allows this type - // of message to be displayed, and unconditionally in update.php. - $message = ''; - if ($this->isErrorDisplayable($error)) { - // If error type is 'User notice' then treat it as debug information - // instead of an error message. - // @see debug() - if ($error['%type'] == 'User notice') { - $error['%type'] = 'Debug'; - } - - $error = $this->simplifyFileInError($error); - - unset($error['backtrace']); - - if (!$this->isErrorLevelVerbose()) { - // Without verbose logging, use a simple message. - - // We call SafeMarkup::format directly here, rather than use t() since - // we are in the middle of error handling, and we don't want t() to - // cause further errors. - $message = SafeMarkup::format('%type: @message in %function (line %line of %file).', $error); - } - else { - // With verbose logging, we will also include a backtrace. - - $backtrace_exception = $exception; - while ($backtrace_exception->getPrevious()) { - $backtrace_exception = $backtrace_exception->getPrevious(); - } - $backtrace = $backtrace_exception->getTrace(); - // First trace is the error itself, already contained in the message. - // While the second trace is the error source and also contained in the - // message, the message doesn't contain argument values, so we output it - // once more in the backtrace. - array_shift($backtrace); - - // Generate a backtrace containing only scalar argument values. - $error['@backtrace'] = Error::formatBacktrace($backtrace); - $message = SafeMarkup::format('%type: @message in %function (line %line of %file).
@backtrace', $error); - } - } - - $content = $this->t('The website encountered an unexpected error. Please try again later.'); - $content .= $message ? '' . $message : ''; - $response = new Response($content, 500, ['Content-Type' => 'text/plain']); - - if ($exception instanceof HttpExceptionInterface) { - $response->setStatusCode($exception->getStatusCode()); - $response->headers->add($exception->getHeaders()); - } - else { - $response->setStatusCode(Response::HTTP_INTERNAL_SERVER_ERROR, '500 Service unavailable (with message)'); - } - - $event->setResponse($response); - } - - /** - * {@inheritdoc} - */ - public static function getSubscribedEvents() { - // Run as the final (very late) KernelEvents::EXCEPTION subscriber. - $events[KernelEvents::EXCEPTION][] = ['onException', -256]; - return $events; - } - - /** - * Checks whether the error level is verbose or not. - * - * @return bool - */ - protected function isErrorLevelVerbose() { - return $this->getErrorLevel() === ERROR_REPORTING_DISPLAY_VERBOSE; - } - - /** - * Wrapper for error_displayable(). - * - * @param $error - * Optional error to examine for ERROR_REPORTING_DISPLAY_SOME. - * - * @return bool - * - * @see \error_displayable - */ - protected function isErrorDisplayable($error) { - return error_displayable($error); - } - - /** - * Attempts to reduce error verbosity in the error message's file path. - * - * Attempts to reduce verbosity by removing DRUPAL_ROOT from the file path in - * the message. This does not happen for (false) security. - * - * @param $error - * Optional error to examine for ERROR_REPORTING_DISPLAY_SOME. - * - * @return - * The updated $error. - */ - protected function simplifyFileInError($error) { - // Attempt to reduce verbosity by removing DRUPAL_ROOT from the file path - // in the message. This does not happen for (false) security. - $root_length = strlen(DRUPAL_ROOT); - if (substr($error['%file'], 0, $root_length) == DRUPAL_ROOT) { - $error['%file'] = substr($error['%file'], $root_length + 1); - } - return $error; - } - -} diff --git a/core/lib/Drupal/Core/Extension/ThemeHandler.php b/core/lib/Drupal/Core/Extension/ThemeHandler.php index d54ff1f..2fd75e1 100644 --- a/core/lib/Drupal/Core/Extension/ThemeHandler.php +++ b/core/lib/Drupal/Core/Extension/ThemeHandler.php @@ -216,16 +216,10 @@ public function addTheme(Extension $theme) { * {@inheritdoc} */ public function refreshInfo() { + $this->reset(); $extension_config = $this->configFactory->get('core.extension'); $installed = $extension_config->get('theme'); - // Only refresh the info if a theme has been installed. Modules are - // installed before themes by the installer and this method is called during - // module installation. - if (empty($installed) && empty($this->list)) { - return; - } - $this->reset(); // @todo Avoid re-scanning all themes by retaining the original (unaltered) // theme info somewhere. $list = $this->rebuildThemeData(); diff --git a/core/lib/Drupal/Core/Field/BaseFieldDefinition.php b/core/lib/Drupal/Core/Field/BaseFieldDefinition.php index 37cc103..c7bb233 100644 --- a/core/lib/Drupal/Core/Field/BaseFieldDefinition.php +++ b/core/lib/Drupal/Core/Field/BaseFieldDefinition.php @@ -89,6 +89,7 @@ public static function createFromFieldStorageDefinition(FieldStorageDefinitionIn ->setLabel($definition->getLabel()) ->setName($definition->getName()) ->setProvider($definition->getProvider()) + ->setQueryable($definition->isQueryable()) ->setRevisionable($definition->isRevisionable()) ->setSettings($definition->getSettings()) ->setTargetEntityTypeId($definition->getTargetEntityTypeId()) @@ -286,8 +287,7 @@ public function isMultiple() { * {@inheritdoc} */ public function isQueryable() { - @trigger_error('BaseFieldDefinition::isQueryable() is deprecated in Drupal 8.4.0 and will be removed before Drupal 9.0.0. Instead, you should use \Drupal\Core\Field\BaseFieldDefinition::hasCustomStorage(). See https://www.drupal.org/node/2856563.', E_USER_DEPRECATED); - return !$this->hasCustomStorage(); + return isset($this->definition['queryable']) ? $this->definition['queryable'] : !$this->isComputed(); } /** @@ -298,14 +298,8 @@ public function isQueryable() { * * @return static * The object itself for chaining. - * - * @deprecated in Drupal 8.4.0 and will be removed before Drupal 9.0.0. Use - * \Drupal\Core\Field\BaseFieldDefinition::setCustomStorage() instead. - * - * @see https://www.drupal.org/node/2856563 */ public function setQueryable($queryable) { - @trigger_error('BaseFieldDefinition::setQueryable() is deprecated in Drupal 8.4.0 and will be removed before Drupal 9.0.0. Instead, you should use \Drupal\Core\Field\BaseFieldDefinition::setCustomStorage(). See https://www.drupal.org/node/2856563.', E_USER_DEPRECATED); $this->definition['queryable'] = $queryable; return $this; } @@ -585,7 +579,7 @@ protected function getFieldItemClass() { public function __sleep() { // Do not serialize the statically cached property definitions. $vars = get_object_vars($this); - unset($vars['propertyDefinitions'], $vars['typedDataManager']); + unset($vars['propertyDefinitions']); return array_keys($vars); } diff --git a/core/lib/Drupal/Core/Field/FieldStorageDefinitionInterface.php b/core/lib/Drupal/Core/Field/FieldStorageDefinitionInterface.php index d703659..1b842b9 100644 --- a/core/lib/Drupal/Core/Field/FieldStorageDefinitionInterface.php +++ b/core/lib/Drupal/Core/Field/FieldStorageDefinitionInterface.php @@ -109,12 +109,6 @@ public function isRevisionable(); * * @return bool * TRUE if the field is queryable. - * - * @deprecated in Drupal 8.4.0 and will be removed before Drupal 9.0.0. Use - * \Drupal\Core\Field\FieldStorageDefinitionInterface::hasCustomStorage() - * instead. - * - * @see https://www.drupal.org/node/2856563 */ public function isQueryable(); diff --git a/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/TimestampItem.php b/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/TimestampItem.php index a5d6f38..8c3ffd2 100644 --- a/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/TimestampItem.php +++ b/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/TimestampItem.php @@ -2,9 +2,8 @@ namespace Drupal\Core\Field\Plugin\Field\FieldType; -use Drupal\Core\Field\FieldDefinitionInterface; -use Drupal\Core\Field\FieldItemBase; use Drupal\Core\Field\FieldStorageDefinitionInterface; +use Drupal\Core\Field\FieldItemBase; use Drupal\Core\TypedData\DataDefinition; /** @@ -54,14 +53,4 @@ public static function schema(FieldStorageDefinitionInterface $field_definition) ]; } - /** - * {@inheritdoc} - */ - public static function generateSampleValue(FieldDefinitionInterface $field_definition) { - // Pick a random timestamp in the past year. - $timestamp = \Drupal::time()->getRequestTime() - mt_rand(0, 86400 * 365); - $values['value'] = $timestamp; - return $values; - } - } diff --git a/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/LanguageSelectWidget.php b/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/LanguageSelectWidget.php index 0872ba3..776e060 100644 --- a/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/LanguageSelectWidget.php +++ b/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/LanguageSelectWidget.php @@ -27,32 +27,7 @@ public function formElement(FieldItemListInterface $items, $delta, array $elemen $element['value'] = $element + [ '#type' => 'language_select', '#default_value' => $items[$delta]->value, - '#languages' => $this->getSetting('include_locked') ? LanguageInterface::STATE_ALL : LanguageInterface::STATE_CONFIGURABLE, - ]; - - return $element; - } - - /** - * {@inheritdoc} - */ - public static function defaultSettings() { - $settings = parent::defaultSettings(); - $settings['include_locked'] = TRUE; - - return $settings; - } - - /** - * {@inheritdoc} - */ - public function settingsForm(array $form, FormStateInterface $form_state) { - $element = parent::settingsForm($form, $form_state); - - $element['include_locked'] = [ - '#type' => 'checkbox', - '#title' => $this->t('Include locked languages such as Not specified and Not applicable'), - '#default_value' => $this->getSetting('include_locked'), + '#languages' => LanguageInterface::STATE_ALL, ]; return $element; diff --git a/core/lib/Drupal/Core/FileTransfer/FTPExtension.php b/core/lib/Drupal/Core/FileTransfer/FTPExtension.php index 9f18963..c5f1b22 100644 --- a/core/lib/Drupal/Core/FileTransfer/FTPExtension.php +++ b/core/lib/Drupal/Core/FileTransfer/FTPExtension.php @@ -45,7 +45,7 @@ protected function createDirectoryJailed($directory) { protected function removeDirectoryJailed($directory) { $pwd = ftp_pwd($this->connection); if (!ftp_chdir($this->connection, $directory)) { - throw new FileTransferException("Unable to change the current directory to @directory", NULL, ['@directory' => $directory]); + throw new FileTransferException("Unable to change to directory @directory", NULL, ['@directory' => $directory]); } $list = @ftp_nlist($this->connection, '.'); if (!$list) { @@ -65,7 +65,7 @@ protected function removeDirectoryJailed($directory) { } ftp_chdir($this->connection, $pwd); if (!ftp_rmdir($this->connection, $directory)) { - throw new FileTransferException("Unable to remove the directory @directory", NULL, ['@directory' => $directory]); + throw new FileTransferException("Unable to remove to directory @directory", NULL, ['@directory' => $directory]); } } @@ -74,7 +74,7 @@ protected function removeDirectoryJailed($directory) { */ protected function removeFileJailed($destination) { if (!ftp_delete($this->connection, $destination)) { - throw new FileTransferException("Unable to remove the file @file", NULL, ['@file' => $destination]); + throw new FileTransferException("Unable to remove to file @file", NULL, ['@file' => $destination]); } } diff --git a/core/lib/Drupal/Core/Form/FormValidator.php b/core/lib/Drupal/Core/Form/FormValidator.php index 8e976f5..7398053 100644 --- a/core/lib/Drupal/Core/Form/FormValidator.php +++ b/core/lib/Drupal/Core/Form/FormValidator.php @@ -229,12 +229,8 @@ protected function finalizeValidation(&$form, FormStateInterface &$form_state, $ * theming, and hook_form_alter functions. */ protected function doValidateForm(&$elements, FormStateInterface &$form_state, $form_id = NULL) { - // Recurse through all children, sorting the elements so that the order of - // error messages displayed to the user matches the order of elements in - // the form. Use a copy of $elements so that it is not modified by the - // sorting itself. - $elements_sorted = $elements; - foreach (Element::children($elements_sorted, TRUE) as $key) { + // Recurse through all children. + foreach (Element::children($elements) as $key) { if (isset($elements[$key]) && $elements[$key]) { $this->doValidateForm($elements[$key], $form_state); } diff --git a/core/lib/Drupal/Core/Layout/LayoutPluginManager.php b/core/lib/Drupal/Core/Layout/LayoutPluginManager.php index 0a28785..e11f105 100644 --- a/core/lib/Drupal/Core/Layout/LayoutPluginManager.php +++ b/core/lib/Drupal/Core/Layout/LayoutPluginManager.php @@ -3,13 +3,13 @@ namespace Drupal\Core\Layout; use Drupal\Component\Annotation\Plugin\Discovery\AnnotationBridgeDecorator; +use Drupal\Component\Plugin\Discovery\DerivativeDiscoveryDecorator; use Drupal\Component\Plugin\Exception\InvalidPluginDefinitionException; use Drupal\Core\Cache\CacheBackendInterface; use Drupal\Core\Extension\ModuleHandlerInterface; use Drupal\Core\Extension\ThemeHandlerInterface; use Drupal\Core\Plugin\DefaultPluginManager; use Drupal\Core\Plugin\Discovery\AnnotatedClassDiscovery; -use Drupal\Core\Plugin\Discovery\ContainerDerivativeDiscoveryDecorator; use Drupal\Core\Plugin\Discovery\YamlDiscoveryDecorator; use Drupal\Core\Layout\Annotation\Layout; @@ -66,7 +66,7 @@ protected function getDiscovery() { $discovery = new AnnotatedClassDiscovery($this->subdir, $this->namespaces, $this->pluginDefinitionAnnotationName, $this->additionalAnnotationNamespaces); $discovery = new YamlDiscoveryDecorator($discovery, 'layouts', $this->moduleHandler->getModuleDirectories() + $this->themeHandler->getThemeDirectories()); $discovery = new AnnotationBridgeDecorator($discovery, $this->pluginDefinitionAnnotationName); - $discovery = new ContainerDerivativeDiscoveryDecorator($discovery); + $discovery = new DerivativeDiscoveryDecorator($discovery); $this->discovery = $discovery; } return $this->discovery; diff --git a/core/lib/Drupal/Core/Plugin/DefaultSingleLazyPluginCollection.php b/core/lib/Drupal/Core/Plugin/DefaultSingleLazyPluginCollection.php index e2bff294..519e93d 100644 --- a/core/lib/Drupal/Core/Plugin/DefaultSingleLazyPluginCollection.php +++ b/core/lib/Drupal/Core/Plugin/DefaultSingleLazyPluginCollection.php @@ -52,7 +52,10 @@ class DefaultSingleLazyPluginCollection extends LazyPluginCollection { */ public function __construct(PluginManagerInterface $manager, $instance_id, array $configuration) { $this->manager = $manager; - $this->addInstanceId($instance_id, $configuration); + $this->instanceId = $instance_id; + // This is still needed by the parent LazyPluginCollection class. + $this->instanceIDs = [$instance_id => $instance_id]; + $this->configuration = $configuration; } /** @@ -92,8 +95,6 @@ public function setConfiguration($configuration) { */ public function addInstanceId($id, $configuration = NULL) { $this->instanceId = $id; - // Reset the list of instance IDs since there can be only one. - $this->instanceIDs = []; parent::addInstanceId($id, $configuration); if ($configuration !== NULL) { $this->setConfiguration($configuration); diff --git a/core/lib/Drupal/Core/TypedData/ComplexDataDefinitionBase.php b/core/lib/Drupal/Core/TypedData/ComplexDataDefinitionBase.php index 0002497..0acdba4 100644 --- a/core/lib/Drupal/Core/TypedData/ComplexDataDefinitionBase.php +++ b/core/lib/Drupal/Core/TypedData/ComplexDataDefinitionBase.php @@ -42,7 +42,7 @@ public function getMainPropertyName() { public function __sleep() { // Do not serialize the cached property definitions. $vars = get_object_vars($this); - unset($vars['propertyDefinitions'], $vars['typedDataManager']); + unset($vars['propertyDefinitions']); return array_keys($vars); } diff --git a/core/lib/Drupal/Core/TypedData/DataDefinition.php b/core/lib/Drupal/Core/TypedData/DataDefinition.php index 52a4394..7eec1a9 100644 --- a/core/lib/Drupal/Core/TypedData/DataDefinition.php +++ b/core/lib/Drupal/Core/TypedData/DataDefinition.php @@ -7,8 +7,6 @@ */ class DataDefinition implements DataDefinitionInterface, \ArrayAccess { - use TypedDataTrait; - /** * The array holding values for all definition keys. * @@ -260,7 +258,7 @@ public function setSetting($setting_name, $value) { */ public function getConstraints() { $constraints = isset($this->definition['constraints']) ? $this->definition['constraints'] : []; - $constraints += $this->getTypedDataManager()->getDefaultConstraints($this); + $constraints += \Drupal::typedDataManager()->getDefaultConstraints($this); return $constraints; } @@ -342,14 +340,4 @@ public function toArray() { return $this->definition; } - /** - * {@inheritdoc} - */ - public function __sleep() { - // Never serialize the typed data manager. - $vars = get_object_vars($this); - unset($vars['typedDataManager']); - return array_keys($vars); - } - } diff --git a/core/lib/Drupal/Core/TypedData/TypedDataManager.php b/core/lib/Drupal/Core/TypedData/TypedDataManager.php index dbf0d5f..8c4f265 100644 --- a/core/lib/Drupal/Core/TypedData/TypedDataManager.php +++ b/core/lib/Drupal/Core/TypedData/TypedDataManager.php @@ -117,13 +117,7 @@ public function createDataDefinition($data_type) { throw new \InvalidArgumentException("Invalid data type '$data_type' has been given"); } $class = $type_definition['definition_class']; - $data_definition = $class::createFromDataType($data_type); - - if (method_exists($data_definition, 'setTypedDataManager')) { - $data_definition->setTypedDataManager($this); - } - - return $data_definition; + return $class::createFromDataType($data_type); } /** diff --git a/core/modules/block/src/Tests/Views/DisplayBlockTest.php b/core/modules/block/src/Tests/Views/DisplayBlockTest.php index 3d35845..64f4851 100644 --- a/core/modules/block/src/Tests/Views/DisplayBlockTest.php +++ b/core/modules/block/src/Tests/Views/DisplayBlockTest.php @@ -286,7 +286,7 @@ public function testBlockEmptyRendering() { $this->drupalGet($url); $this->assertEqual(0, count($this->xpath('//div[contains(@class, "block-views-blocktest-view-block-block-1")]'))); - // Ensure that the view cacheability metadata is propagated even, for an + // Ensure that the view cachability metadata is propagated even, for an // empty block. $this->assertCacheTags(array_merge($block->getCacheTags(), ['block_view', 'config:block_list', 'config:views.view.test_view_block' , 'http_response', 'rendered'])); $this->assertCacheContexts(['url.query_args:_wrapper_format']); diff --git a/core/modules/block_content/src/Entity/BlockContent.php b/core/modules/block_content/src/Entity/BlockContent.php index b7c3d4d..9d0bf74 100644 --- a/core/modules/block_content/src/Entity/BlockContent.php +++ b/core/modules/block_content/src/Entity/BlockContent.php @@ -52,11 +52,6 @@ * "langcode" = "langcode", * "uuid" = "uuid" * }, - * revision_metadata_keys = { - * "revision_user" = "revision_user", - * "revision_created" = "revision_created", - * "revision_log_message" = "revision_log" - * }, * bundle_entity_type = "block_content_type", * field_ui_base_route = "entity.block_content_type.edit_form", * render_cache = FALSE, diff --git a/core/modules/comment/comment.module b/core/modules/comment/comment.module index 489a391..ee26510 100644 --- a/core/modules/comment/comment.module +++ b/core/modules/comment/comment.module @@ -353,7 +353,8 @@ function comment_entity_storage_load($entities, $entity_type) { if (!\Drupal::entityManager()->getDefinition($entity_type)->entityClassImplements(FieldableEntityInterface::class)) { return; } - if (!\Drupal::service('comment.manager')->getFields($entity_type)) { + // @todo Investigate in https://www.drupal.org/node/2527866 why we need that. + if (!\Drupal::hasService('comment.manager') || !\Drupal::service('comment.manager')->getFields($entity_type)) { // Do not query database when entity has no comment fields. return; } diff --git a/core/modules/comment/src/Plugin/views/argument/UserUid.php b/core/modules/comment/src/Plugin/views/argument/UserUid.php index 7023102..69621559 100644 --- a/core/modules/comment/src/Plugin/views/argument/UserUid.php +++ b/core/modules/comment/src/Plugin/views/argument/UserUid.php @@ -3,7 +3,6 @@ namespace Drupal\comment\Plugin\views\argument; use Drupal\Core\Database\Connection; -use Drupal\Core\Database\Query\Condition; use Drupal\views\Plugin\views\argument\ArgumentPluginBase; use Symfony\Component\DependencyInjection\ContainerInterface; @@ -91,7 +90,7 @@ public function query($group_by = FALSE) { $subselect->where("c.entity_id = $this->tableAlias.$entity_id"); $subselect->condition('c.entity_type', $entity_type); - $condition = (new Condition('OR')) + $condition = db_or() ->condition("$this->tableAlias.uid", $this->argument, '=') ->exists($subselect); diff --git a/core/modules/comment/src/Plugin/views/filter/UserUid.php b/core/modules/comment/src/Plugin/views/filter/UserUid.php index 248b460..adee320 100644 --- a/core/modules/comment/src/Plugin/views/filter/UserUid.php +++ b/core/modules/comment/src/Plugin/views/filter/UserUid.php @@ -2,7 +2,6 @@ namespace Drupal\comment\Plugin\views\filter; -use Drupal\Core\Database\Query\Condition; use Drupal\views\Plugin\views\filter\FilterPluginBase; /** @@ -27,7 +26,7 @@ public function query() { $subselect->where("c.entity_id = $this->tableAlias.$entity_id"); $subselect->condition('c.entity_type', $entity_type); - $condition = (new Condition('OR')) + $condition = db_or() ->condition("$this->tableAlias.uid", $this->value, $this->operator) ->exists($subselect); diff --git a/core/modules/config/tests/config_schema_test/config/schema/config_schema_test.schema.yml b/core/modules/config/tests/config_schema_test/config/schema/config_schema_test.schema.yml index ce894c2..c10e268 100644 --- a/core/modules/config/tests/config_schema_test/config/schema/config_schema_test.schema.yml +++ b/core/modules/config/tests/config_schema_test/config/schema/config_schema_test.schema.yml @@ -297,41 +297,3 @@ test.double_brackets.breed: mapping: breed: type: string - -config_schema_test.schema_sequence_sort: - type: config_object - mapping: - keyed_sort: - type: sequence - orderby: key - sequence: - - type: string - value_sort: - type: sequence - orderby: value - sequence: - - type: string - no_sort: - type: sequence - sequence: - - type: string - complex_sort_value: - type: sequence - orderby: value - sequence: - - type: mapping - mapping: - foo: - type: string - bar: - type: string - complex_sort_key: - type: sequence - orderby: key - sequence: - - type: mapping - mapping: - foo: - type: string - bar: - type: string diff --git a/core/modules/config/tests/config_test/config/install/config_test.validation.yml b/core/modules/config/tests/config_test/config/install/config_test.validation.yml deleted file mode 100644 index 3741468..0000000 --- a/core/modules/config/tests/config_test/config/install/config_test.validation.yml +++ /dev/null @@ -1,7 +0,0 @@ -llama: llama -cat: - type: kitten - count: 2 -giraffe: - hum1: hum1 - hum2: hum2 diff --git a/core/modules/config/tests/config_test/config/schema/config_test.schema.yml b/core/modules/config/tests/config_test/config/schema/config_test.schema.yml index 2cd68a4..e5b75d1 100644 --- a/core/modules/config/tests/config_test/config/schema/config_test.schema.yml +++ b/core/modules/config/tests/config_test/config/schema/config_test.schema.yml @@ -158,39 +158,3 @@ config_test.foo: config_test.bar: type: config_test.foo - -config_test.validation: - type: config_object - label: 'Configuration type' - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateMapping] - mapping: - llama: - type: string - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateLlama] - cat: - type: mapping - mapping: - type: - type: string - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateCats] - count: - type: integer - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateCatCount] - giraffe: - type: sequence - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateSequence] - sequence: - type: string - constraints: - Callback: - callback: [\Drupal\config_test\ConfigValidation, validateGiraffes] diff --git a/core/modules/config/tests/config_test/src/ConfigValidation.php b/core/modules/config/tests/config_test/src/ConfigValidation.php deleted file mode 100644 index f9493b6..0000000 --- a/core/modules/config/tests/config_test/src/ConfigValidation.php +++ /dev/null @@ -1,96 +0,0 @@ -addViolation('no valid llama'); - } - } - - /** - * Validates cats. - * - * @param string $string - * The string to validate. - * @param \Symfony\Component\Validator\Context\ExecutionContextInterface $context - * The validation execution context. - */ - public static function validateCats($string, ExecutionContextInterface $context) { - if (!in_array($string, ['kitten', 'cats', 'nyans'])) { - $context->addViolation('no valid cat'); - } - } - - /** - * Validates a number. - * - * @param int $count - * The integer to validate. - * @param \Symfony\Component\Validator\Context\ExecutionContextInterface $context - * The validation execution context. - */ - public static function validateCatCount($count, ExecutionContextInterface $context) { - if ($count <= 1) { - $context->addViolation('no enough cats'); - } - } - - /** - * Validates giraffes. - * - * @param string $string - * The string to validate. - * @param \Symfony\Component\Validator\Context\ExecutionContextInterface $context - * The validation execution context. - */ - public static function validateGiraffes($string, ExecutionContextInterface $context) { - if (strpos($string, 'hum') !== 0) { - $context->addViolation('Giraffes just hum'); - } - } - - /** - * Validates a mapping. - * - * @param array $mapping - * The data to validate. - * @param \Symfony\Component\Validator\Context\ExecutionContextInterface $context - * The validation execution context. - */ - public static function validateMapping($mapping, ExecutionContextInterface $context) { - if ($diff = array_diff(array_keys($mapping), ['llama', 'cat', 'giraffe', '_core'])) { - $context->addViolation('Missing giraffe.'); - } - } - - /** - * Validates a sequence. - * - * @param array $sequence - * The data to validate. - * @param \Symfony\Component\Validator\Context\ExecutionContextInterface $context - * The validation execution context. - */ - public static function validateSequence($sequence, ExecutionContextInterface $context) { - if (isset($sequence['invalid-key'])) { - $context->addViolation('Invalid giraffe key.'); - } - } - -} diff --git a/core/modules/content_moderation/tests/src/Unit/ContentPreprocessTest.php b/core/modules/content_moderation/tests/src/Unit/ContentPreprocessTest.php index 8084716..5de0b2a 100644 --- a/core/modules/content_moderation/tests/src/Unit/ContentPreprocessTest.php +++ b/core/modules/content_moderation/tests/src/Unit/ContentPreprocessTest.php @@ -5,14 +5,13 @@ use Drupal\content_moderation\ContentPreprocess; use Drupal\Core\Routing\CurrentRouteMatch; use Drupal\node\Entity\Node; -use Drupal\Tests\UnitTestCase; /** * @coversDefaultClass \Drupal\content_moderation\ContentPreprocess * * @group content_moderation */ -class ContentPreprocessTest extends UnitTestCase { +class ContentPreprocessTest extends \PHPUnit_Framework_TestCase { /** * @covers ::isLatestVersionPage diff --git a/core/modules/content_moderation/tests/src/Unit/LatestRevisionCheckTest.php b/core/modules/content_moderation/tests/src/Unit/LatestRevisionCheckTest.php index 412a0ea..1f8838b 100644 --- a/core/modules/content_moderation/tests/src/Unit/LatestRevisionCheckTest.php +++ b/core/modules/content_moderation/tests/src/Unit/LatestRevisionCheckTest.php @@ -9,14 +9,13 @@ use Drupal\node\Entity\Node; use Drupal\content_moderation\Access\LatestRevisionCheck; use Drupal\content_moderation\ModerationInformation; -use Drupal\Tests\UnitTestCase; use Symfony\Component\Routing\Route; /** * @coversDefaultClass \Drupal\content_moderation\Access\LatestRevisionCheck * @group content_moderation */ -class LatestRevisionCheckTest extends UnitTestCase { +class LatestRevisionCheckTest extends \PHPUnit_Framework_TestCase { /** * Test the access check of the LatestRevisionCheck service. diff --git a/core/modules/content_moderation/tests/src/Unit/ModerationInformationTest.php b/core/modules/content_moderation/tests/src/Unit/ModerationInformationTest.php index 57fe761..8d319ef 100644 --- a/core/modules/content_moderation/tests/src/Unit/ModerationInformationTest.php +++ b/core/modules/content_moderation/tests/src/Unit/ModerationInformationTest.php @@ -10,14 +10,13 @@ use Drupal\Core\Entity\EntityTypeManagerInterface; use Drupal\Core\Session\AccountInterface; use Drupal\content_moderation\ModerationInformation; -use Drupal\Tests\UnitTestCase; use Drupal\workflows\WorkflowInterface; /** * @coversDefaultClass \Drupal\content_moderation\ModerationInformation * @group content_moderation */ -class ModerationInformationTest extends UnitTestCase { +class ModerationInformationTest extends \PHPUnit_Framework_TestCase { /** * Builds a mock user. diff --git a/core/modules/content_moderation/tests/src/Unit/StateTransitionValidationTest.php b/core/modules/content_moderation/tests/src/Unit/StateTransitionValidationTest.php index b7c3942..2e03447 100644 --- a/core/modules/content_moderation/tests/src/Unit/StateTransitionValidationTest.php +++ b/core/modules/content_moderation/tests/src/Unit/StateTransitionValidationTest.php @@ -7,7 +7,6 @@ use Drupal\Core\Entity\ContentEntityInterface; use Drupal\Core\Session\AccountInterface; use Drupal\content_moderation\StateTransitionValidation; -use Drupal\Tests\UnitTestCase; use Drupal\workflows\Entity\Workflow; use Drupal\workflows\WorkflowTypeInterface; use Drupal\workflows\WorkflowTypeManager; @@ -17,7 +16,7 @@ * @coversDefaultClass \Drupal\content_moderation\StateTransitionValidation * @group content_moderation */ -class StateTransitionValidationTest extends UnitTestCase { +class StateTransitionValidationTest extends \PHPUnit_Framework_TestCase { /** * Verifies user-aware transition validation. @@ -63,9 +62,6 @@ protected function setUpModerationInformation(ContentEntityInterface $entity) { // mocked. $container = new ContainerBuilder(); $workflow_type = $this->prophesize(WorkflowTypeInterface::class); - $workflow_type->setConfiguration(Argument::any())->will(function ($arguments) { - $this->getConfiguration()->willReturn($arguments[0]); - }); $workflow_type->decorateState(Argument::any())->willReturnArgument(0); $workflow_type->decorateTransition(Argument::any())->willReturnArgument(0); $workflow_manager = $this->prophesize(WorkflowTypeManager::class); diff --git a/core/modules/contextual/contextual.module b/core/modules/contextual/contextual.module index a814c39..cf20318 100644 --- a/core/modules/contextual/contextual.module +++ b/core/modules/contextual/contextual.module @@ -36,7 +36,6 @@ function contextual_toolbar() { '#attributes' => [ 'class' => ['toolbar-icon', 'toolbar-icon-edit'], 'aria-pressed' => 'false', - 'type' => 'button', ], ], '#wrapper_attributes' => [ diff --git a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeCustomFormatter.php b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeCustomFormatter.php index 984423d..445b7ee 100644 --- a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeCustomFormatter.php +++ b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeCustomFormatter.php @@ -32,18 +32,30 @@ public static function defaultSettings() { * {@inheritdoc} */ public function viewElements(FieldItemListInterface $items, $langcode) { - // @todo Evaluate removing this method in - // https://www.drupal.org/node/2793143 to determine if the behavior and - // markup in the base class implementation can be used instead. $elements = []; foreach ($items as $delta => $item) { + $output = ''; if (!empty($item->date)) { /** @var \Drupal\Core\Datetime\DrupalDateTime $date */ $date = $item->date; - $elements[$delta] = $this->buildDate($date); + if ($this->getFieldSetting('datetime_type') == 'date') { + // A date without time will pick up the current time, use the default. + datetime_date_default_time($date); + } + $this->setTimeZone($date); + + $output = $this->formatDate($date); } + $elements[$delta] = [ + '#markup' => $output, + '#cache' => [ + 'contexts' => [ + 'timezone', + ], + ], + ]; } return $elements; diff --git a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeDefaultFormatter.php b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeDefaultFormatter.php index 0b75b4e..7ddecec 100644 --- a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeDefaultFormatter.php +++ b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeDefaultFormatter.php @@ -3,6 +3,7 @@ namespace Drupal\datetime\Plugin\Field\FieldFormatter; use Drupal\Core\Datetime\DrupalDateTime; +use Drupal\Core\Field\FieldItemListInterface; use Drupal\Core\Form\FormStateInterface; /** @@ -30,6 +31,59 @@ public static function defaultSettings() { /** * {@inheritdoc} */ + public function viewElements(FieldItemListInterface $items, $langcode) { + $elements = []; + + foreach ($items as $delta => $item) { + $output = ''; + $iso_date = ''; + + if ($item->date) { + /** @var \Drupal\Core\Datetime\DrupalDateTime $date */ + $date = $item->date; + + if ($this->getFieldSetting('datetime_type') == 'date') { + // A date without time will pick up the current time, use the default. + datetime_date_default_time($date); + } + + // Create the ISO date in Universal Time. + $iso_date = $date->format("Y-m-d\TH:i:s") . 'Z'; + + $this->setTimeZone($date); + + $output = $this->formatDate($date); + } + + // Display the date using theme datetime. + $elements[$delta] = [ + '#cache' => [ + 'contexts' => [ + 'timezone', + ], + ], + '#theme' => 'time', + '#text' => $output, + '#html' => FALSE, + '#attributes' => [ + 'datetime' => $iso_date, + ], + ]; + if (!empty($item->_attributes)) { + $elements[$delta]['#attributes'] += $item->_attributes; + // Unset field item attributes since they have been included in the + // formatter output and should not be rendered in the field template. + unset($item->_attributes); + } + } + + return $elements; + + } + + /** + * {@inheritdoc} + */ protected function formatDate($date) { $format_type = $this->getSetting('format_type'); $timezone = $this->getSetting('timezone_override') ?: $date->getTimezone()->getName(); diff --git a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeFormatterBase.php b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeFormatterBase.php index 71b9467..f4e5ff5 100644 --- a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeFormatterBase.php +++ b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimeFormatterBase.php @@ -6,7 +6,6 @@ use Drupal\Core\Datetime\DrupalDateTime; use Drupal\Core\Entity\EntityStorageInterface; use Drupal\Core\Field\FieldDefinitionInterface; -use Drupal\Core\Field\FieldItemListInterface; use Drupal\Core\Field\FormatterBase; use Drupal\Core\Form\FormStateInterface; use Drupal\Core\Plugin\ContainerFactoryPluginInterface; @@ -119,30 +118,6 @@ public function settingsSummary() { } /** - * {@inheritdoc} - */ - public function viewElements(FieldItemListInterface $items, $langcode) { - $elements = []; - - foreach ($items as $delta => $item) { - if ($item->date) { - /** @var \Drupal\Core\Datetime\DrupalDateTime $date */ - $date = $item->date; - $elements[$delta] = $this->buildDateWithIsoAttribute($date); - - if (!empty($item->_attributes)) { - $elements[$delta]['#attributes'] += $item->_attributes; - // Unset field item attributes since they have been included in the - // formatter output and should not be rendered in the field template. - unset($item->_attributes); - } - } - } - - return $elements; - } - - /** * Creates a formatted date value as a string. * * @param object $date @@ -192,69 +167,4 @@ protected function getFormatSettings() { return $settings; } - /** - * Creates a render array from a date object. - * - * @param \Drupal\Core\Datetime\DrupalDateTime $date - * A date object. - * - * @return array - * A render array. - */ - protected function buildDate(DrupalDateTime $date) { - if ($this->getFieldSetting('datetime_type') == DateTimeItem::DATETIME_TYPE_DATE) { - // A date without time will pick up the current time, use the default. - datetime_date_default_time($date); - } - $this->setTimeZone($date); - - $build = [ - '#markup' => $this->formatDate($date), - '#cache' => [ - 'contexts' => [ - 'timezone', - ], - ], - ]; - - return $build; - } - - /** - * Creates a render array from a date object with ISO date attribute. - * - * @param \Drupal\Core\Datetime\DrupalDateTime $date - * A date object. - * - * @return array - * A render array. - */ - protected function buildDateWithIsoAttribute(DrupalDateTime $date) { - if ($this->getFieldSetting('datetime_type') == DateTimeItem::DATETIME_TYPE_DATE) { - // A date without time will pick up the current time, use the default. - datetime_date_default_time($date); - } - - // Create the ISO date in Universal Time. - $iso_date = $date->format("Y-m-d\TH:i:s") . 'Z'; - - $this->setTimeZone($date); - - $build = [ - '#theme' => 'time', - '#text' => $this->formatDate($date), - '#html' => FALSE, - '#attributes' => [ - 'datetime' => $iso_date, - ], - '#cache' => [ - 'contexts' => [ - 'timezone', - ], - ], - ]; - - return $build; - } - } diff --git a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimePlainFormatter.php b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimePlainFormatter.php index 7f0dee2..d35d380 100644 --- a/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimePlainFormatter.php +++ b/core/modules/datetime/src/Plugin/Field/FieldFormatter/DateTimePlainFormatter.php @@ -25,12 +25,27 @@ public function viewElements(FieldItemListInterface $items, $langcode) { $elements = []; foreach ($items as $delta => $item) { + $output = ''; if (!empty($item->date)) { /** @var \Drupal\Core\Datetime\DrupalDateTime $date */ $date = $item->date; - $elements[$delta] = $this->buildDate($date); + if ($this->getFieldSetting('datetime_type') == 'date') { + // A date without time will pick up the current time, use the default. + datetime_date_default_time($date); + } + $this->setTimeZone($date); + + $output = $this->formatDate($date); } + $elements[$delta] = [ + '#cache' => [ + 'contexts' => [ + 'timezone', + ], + ], + '#markup' => $output, + ]; } return $elements; diff --git a/core/modules/datetime_range/src/DateTimeRangeTrait.php b/core/modules/datetime_range/src/DateTimeRangeTrait.php index 3f05b82..5a34f2c 100644 --- a/core/modules/datetime_range/src/DateTimeRangeTrait.php +++ b/core/modules/datetime_range/src/DateTimeRangeTrait.php @@ -2,7 +2,8 @@ namespace Drupal\datetime_range; -use Drupal\Core\Field\FieldItemListInterface; +use Drupal\Core\Datetime\DrupalDateTime; +use Drupal\datetime\Plugin\Field\FieldType\DateTimeItem; /** * Provides friendly methods for datetime range. @@ -10,40 +11,68 @@ trait DateTimeRangeTrait { /** - * {@inheritdoc} + * Creates a render array from a date object. + * + * @param \Drupal\Core\Datetime\DrupalDateTime $date + * A date object. + * + * @return array + * A render array. */ - public function viewElements(FieldItemListInterface $items, $langcode) { - $elements = []; - $separator = $this->getSetting('separator'); - - foreach ($items as $delta => $item) { - if (!empty($item->start_date) && !empty($item->end_date)) { - /** @var \Drupal\Core\Datetime\DrupalDateTime $start_date */ - $start_date = $item->start_date; - /** @var \Drupal\Core\Datetime\DrupalDateTime $end_date */ - $end_date = $item->end_date; - - if ($start_date->getTimestamp() !== $end_date->getTimestamp()) { - $elements[$delta] = [ - 'start_date' => $this->buildDateWithIsoAttribute($start_date), - 'separator' => ['#plain_text' => ' ' . $separator . ' '], - 'end_date' => $this->buildDateWithIsoAttribute($end_date), - ]; - } - else { - $elements[$delta] = $this->buildDateWithIsoAttribute($start_date); - - if (!empty($item->_attributes)) { - $elements[$delta]['#attributes'] += $item->_attributes; - // Unset field item attributes since they have been included in the - // formatter output and should not be rendered in the field template. - unset($item->_attributes); - } - } - } + protected function buildDate(DrupalDateTime $date) { + if ($this->getFieldSetting('datetime_type') == DateTimeItem::DATETIME_TYPE_DATE) { + // A date without time will pick up the current time, use the default. + datetime_date_default_time($date); } + $this->setTimeZone($date); - return $elements; + $build = [ + '#plain_text' => $this->formatDate($date), + '#cache' => [ + 'contexts' => [ + 'timezone', + ], + ], + ]; + + return $build; + } + + /** + * Creates a render array from a date object with ISO date attribute. + * + * @param \Drupal\Core\Datetime\DrupalDateTime $date + * A date object. + * + * @return array + * A render array. + */ + protected function buildDateWithIsoAttribute(DrupalDateTime $date) { + if ($this->getFieldSetting('datetime_type') == DateTimeItem::DATETIME_TYPE_DATE) { + // A date without time will pick up the current time, use the default. + datetime_date_default_time($date); + } + + // Create the ISO date in Universal Time. + $iso_date = $date->format("Y-m-d\TH:i:s") . 'Z'; + + $this->setTimeZone($date); + + $build = [ + '#theme' => 'time', + '#text' => $this->formatDate($date), + '#html' => FALSE, + '#attributes' => [ + 'datetime' => $iso_date, + ], + '#cache' => [ + 'contexts' => [ + 'timezone', + ], + ], + ]; + + return $build; } } diff --git a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeCustomFormatter.php b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeCustomFormatter.php index 78aa8aa..a26bbef 100644 --- a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeCustomFormatter.php +++ b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeCustomFormatter.php @@ -38,9 +38,6 @@ public static function defaultSettings() { * {@inheritdoc} */ public function viewElements(FieldItemListInterface $items, $langcode) { - // @todo Evaluate removing this method in - // https://www.drupal.org/node/2793143 to determine if the behavior and - // markup in the base class implementation can be used instead. $elements = []; $separator = $this->getSetting('separator'); diff --git a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeDefaultFormatter.php b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeDefaultFormatter.php index b93851b..f07e834 100644 --- a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeDefaultFormatter.php +++ b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangeDefaultFormatter.php @@ -2,6 +2,7 @@ namespace Drupal\datetime_range\Plugin\Field\FieldFormatter; +use Drupal\Core\Field\FieldItemListInterface; use Drupal\Core\Form\FormStateInterface; use Drupal\datetime\Plugin\Field\FieldFormatter\DateTimeDefaultFormatter; use Drupal\datetime_range\DateTimeRangeTrait; @@ -37,6 +38,42 @@ public static function defaultSettings() { /** * {@inheritdoc} */ + public function viewElements(FieldItemListInterface $items, $langcode) { + $elements = []; + $separator = $this->getSetting('separator'); + + foreach ($items as $delta => $item) { + if (!empty($item->start_date) && !empty($item->end_date)) { + /** @var \Drupal\Core\Datetime\DrupalDateTime $start_date */ + $start_date = $item->start_date; + /** @var \Drupal\Core\Datetime\DrupalDateTime $end_date */ + $end_date = $item->end_date; + + if ($start_date->getTimestamp() !== $end_date->getTimestamp()) { + $elements[$delta] = [ + 'start_date' => $this->buildDateWithIsoAttribute($start_date), + 'separator' => ['#plain_text' => ' ' . $separator . ' '], + 'end_date' => $this->buildDateWithIsoAttribute($end_date), + ]; + } + else { + $elements[$delta] = $this->buildDateWithIsoAttribute($start_date); + if (!empty($item->_attributes)) { + $elements[$delta]['#attributes'] += $item->_attributes; + // Unset field item attributes since they have been included in the + // formatter output and should not be rendered in the field template. + unset($item->_attributes); + } + } + } + } + + return $elements; + } + + /** + * {@inheritdoc} + */ public function settingsForm(array $form, FormStateInterface $form_state) { $form = parent::settingsForm($form, $form_state); diff --git a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangePlainFormatter.php b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangePlainFormatter.php index 0f74fba..1fb4702 100644 --- a/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangePlainFormatter.php +++ b/core/modules/datetime_range/src/Plugin/Field/FieldFormatter/DateRangePlainFormatter.php @@ -57,13 +57,6 @@ public function viewElements(FieldItemListInterface $items, $langcode) { } else { $elements[$delta] = $this->buildDate($start_date); - - if (!empty($item->_attributes)) { - $elements[$delta]['#attributes'] += $item->_attributes; - // Unset field item attributes since they have been included in the - // formatter output and should not be rendered in the field template. - unset($item->_attributes); - } } } } diff --git a/core/modules/datetime_range/tests/src/Functional/DateRangeFieldTest.php b/core/modules/datetime_range/tests/src/Functional/DateRangeFieldTest.php index 23ba8fa..ebeb4b8 100644 --- a/core/modules/datetime_range/tests/src/Functional/DateRangeFieldTest.php +++ b/core/modules/datetime_range/tests/src/Functional/DateRangeFieldTest.php @@ -170,16 +170,6 @@ public function testDateRangeField() { $output = $this->renderTestEntity($id); $this->assertContains($expected, $output, new FormattableMarkup('Formatted date field using daterange_custom format displayed as %expected.', ['%expected' => $expected])); - // Test that allowed markup in custom format is preserved and XSS is - // removed. - $this->displayOptions['settings']['date_format'] = '\\<\\s\\t\\r\\o\\n\\g\\>m/d/Y\\<\\/\\s\\t\\r\\o\\n\\g\\>\\<\\s\\c\\r\\i\\p\\t\\>\\a\\l\\e\\r\\t\\(\\S\\t\\r\\i\\n\\g\\.\\f\\r\\o\\m\\C\\h\\a\\r\\C\\o\\d\\e\\(\\8\\8\\,\\8\\3\\,\\8\\3\\)\\)\\<\\/\\s\\c\\r\\i\\p\\t\\>'; - entity_get_display($this->field->getTargetEntityTypeId(), $this->field->getTargetBundle(), 'full') - ->setComponent($field_name, $this->displayOptions) - ->save(); - $expected = '' . $start_date->format('m/d/Y') . 'alert(String.fromCharCode(88,83,83)) - ' . $end_date->format('m/d/Y') . 'alert(String.fromCharCode(88,83,83))'; - $output = $this->renderTestEntity($id); - $this->assertContains($expected, $output, new FormattableMarkup('Formatted date field using daterange_custom format displayed as %expected.', ['%expected' => $expected])); - // Test formatters when start date and end date are the same $this->drupalGet('entity_test/add'); $value = '2012-12-31 00:00:00'; diff --git a/core/modules/editor/src/Tests/QuickEditIntegrationLoadingTest.php b/core/modules/editor/src/Tests/QuickEditIntegrationLoadingTest.php index a4515d5..7141687 100644 --- a/core/modules/editor/src/Tests/QuickEditIntegrationLoadingTest.php +++ b/core/modules/editor/src/Tests/QuickEditIntegrationLoadingTest.php @@ -89,12 +89,12 @@ public function testUsersWithoutPermission() { $response = $this->drupalPost('editor/' . 'node/1/body/en/full', '', [], ['query' => [MainContentViewSubscriber::WRAPPER_FORMAT => 'drupal_ajax']]); $this->assertResponse(403); if (!$user->hasPermission('access in-place editing')) { - $message = "The 'access in-place editing' permission is required."; + $message = "A fatal error occurred: The 'access in-place editing' permission is required."; + $this->assertIdentical(Json::encode(['message' => $message]), $response); } else { - $message = ''; + $this->assertIdentical('{}', $response); } - $this->assertIdentical(Json::encode(['message' => $message]), $response); } } diff --git a/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php b/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php index 276f559..256d271 100644 --- a/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php +++ b/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php @@ -9,7 +9,7 @@ use Drupal\entity_test\Entity\EntityTest; use Drupal\quickedit\MetadataGenerator; use Drupal\Tests\quickedit\Kernel\QuickEditTestBase; -use Drupal\quickedit_test\MockQuickEditEntityFieldAccessCheck; +use Drupal\quickedit_test\MockEditEntityFieldAccessCheck; use Drupal\editor\EditorController; use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpKernel\Event\FilterResponseEvent; @@ -52,7 +52,7 @@ class QuickEditIntegrationTest extends QuickEditTestBase { /** * The access checker object to be used by the metadata generator object. * - * @var \Drupal\quickedit\Access\QuickEditEntityFieldAccessCheckInterface + * @var \Drupal\quickedit\Access\EditEntityFieldAccessCheckInterface */ protected $accessChecker; @@ -165,7 +165,7 @@ public function testEditorSelection() { */ public function testMetadata() { $this->editorManager = $this->container->get('plugin.manager.quickedit.editor'); - $this->accessChecker = new MockQuickEditEntityFieldAccessCheck(); + $this->accessChecker = new MockEditEntityFieldAccessCheck(); $this->editorSelector = $this->container->get('quickedit.editor.selector'); $this->metadataGenerator = new MetadataGenerator($this->accessChecker, $this->editorSelector, $this->editorManager); diff --git a/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php b/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php index 2b0be85..31d6503 100644 --- a/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php +++ b/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php @@ -34,7 +34,7 @@ class EntityReferenceRelationshipTest extends ViewsKernelTestBase { 'test_entity_reference_entity_test_mul_view', 'test_entity_reference_reverse_entity_test_mul_view', 'test_entity_reference_group_by_empty_relationships', - ]; + ]; /** * Modules to install. diff --git a/core/modules/field/tests/src/Kernel/Timestamp/TimestampItemTest.php b/core/modules/field/tests/src/Kernel/Timestamp/TimestampItemTest.php deleted file mode 100644 index 0aaa8d3..0000000 --- a/core/modules/field/tests/src/Kernel/Timestamp/TimestampItemTest.php +++ /dev/null @@ -1,91 +0,0 @@ -fieldStorage = FieldStorageConfig::create([ - 'field_name' => 'field_timestamp', - 'type' => 'timestamp', - 'entity_type' => 'entity_test', - ]); - $this->fieldStorage->save(); - $this->field = FieldConfig::create([ - 'field_storage' => $this->fieldStorage, - 'bundle' => 'entity_test', - ]); - $this->field->save(); - } - - /** - * Tests using entity fields of the datetime field type. - */ - public function testDateTime() { - // Verify entity creation. - $entity = EntityTest::create(); - $value = 1488914208; - $entity->field_timestamp = $value; - $entity->name->value = $this->randomMachineName(); - $this->entityValidateAndSave($entity); - - // Verify entity has been created properly. - $id = $entity->id(); - $entity = EntityTest::load($id); - $this->assertTrue($entity->field_timestamp instanceof FieldItemListInterface, 'Field implements interface.'); - $this->assertTrue($entity->field_timestamp[0] instanceof FieldItemInterface, 'Field item implements interface.'); - $this->assertEquals($entity->field_timestamp->value, $value); - $this->assertEquals($entity->field_timestamp[0]->value, $value); - - // Verify changing the date value. - $new_value = 1488914000; - $entity->field_timestamp->value = $new_value; - $this->assertEquals($entity->field_timestamp->value, $new_value); - - // Read changed entity and assert changed values. - $this->entityValidateAndSave($entity); - $entity = EntityTest::load($id); - $this->assertEquals($entity->field_timestamp->value, $new_value); - - // Test sample item generation. - $entity = EntityTest::create(); - $entity->field_timestamp->generateSampleItems(); - $this->entityValidateAndSave($entity); - - // Ensure there is sample value a generated for the field. - $this->assertNotNull($entity->field_timestamp->value); - } - -} diff --git a/core/modules/field_ui/src/Tests/FieldUIDeleteTest.php b/core/modules/field_ui/src/Tests/FieldUIDeleteTest.php index 631d336..2704157 100644 --- a/core/modules/field_ui/src/Tests/FieldUIDeleteTest.php +++ b/core/modules/field_ui/src/Tests/FieldUIDeleteTest.php @@ -5,7 +5,6 @@ use Drupal\field\Entity\FieldConfig; use Drupal\field\Entity\FieldStorageConfig; use Drupal\simpletest\WebTestBase; -use Drupal\views\Entity\View; use Drupal\views\Tests\ViewTestData; /** @@ -75,13 +74,6 @@ public function testDeleteField() { \Drupal::service('module_installer')->install(['views']); ViewTestData::createTestViews(get_class($this), ['field_test_views']); - $view = View::load('test_view_field_delete'); - $this->assertNotNull($view); - $this->assertTrue($view->status()); - // Test that the View depends on the field. - $dependencies = $view->getDependencies() + ['config' => []]; - $this->assertTrue(in_array("field.storage.node.$field_name", $dependencies['config'])); - // Check the config dependencies of the first field, the field storage must // not be shown as being deleted yet. $this->drupalGet("$bundle_path1/fields/node.$type_name1.$field_name/delete"); @@ -99,13 +91,13 @@ public function testDeleteField() { // Check the config dependencies of the first field. $this->drupalGet("$bundle_path2/fields/node.$type_name2.$field_name/delete"); - $this->assertText(t('The listed configuration will be updated.')); + $this->assertText(t('The listed configuration will be deleted.')); $this->assertText(t('View')); $this->assertText('test_view_field_delete'); $xml = $this->cssSelect('#edit-entity-deletes'); - // Test that nothing is scheduled for deletion. - $this->assertFalse(isset($xml[0]), 'The field currently being deleted is not shown in the entity deletions.'); + // Remove the wrapping HTML. + $this->assertIdentical(FALSE, strpos($xml[0]->asXml(), $field_label), 'The currently being deleted field is not shown in the entity deletions.'); // Delete the second field. $this->fieldUIDeleteField($bundle_path2, "node.$type_name2.$field_name", $field_label, $type_name2); @@ -114,14 +106,6 @@ public function testDeleteField() { $this->assertNull(FieldConfig::loadByName('node', $type_name2, $field_name), 'Field was deleted.'); // Check that the field storage was deleted too. $this->assertNull(FieldStorageConfig::loadByName('node', $field_name), 'Field storage was deleted.'); - - // Test that the View isn't deleted and has been disabled. - $view = View::load('test_view_field_delete'); - $this->assertNotNull($view); - $this->assertFalse($view->status()); - // Test that the View no longer depends on the deleted field. - $dependencies = $view->getDependencies() + ['config' => []]; - $this->assertFalse(in_array("field.storage.node.$field_name", $dependencies['config'])); } } diff --git a/core/modules/file/tests/src/Functional/FileFieldTestBase.php b/core/modules/file/tests/src/Functional/FileFieldTestBase.php index d609687..9ed447a 100644 --- a/core/modules/file/tests/src/Functional/FileFieldTestBase.php +++ b/core/modules/file/tests/src/Functional/FileFieldTestBase.php @@ -259,7 +259,7 @@ public function replaceNodeFile($file, $field_name, $nid, $new_revision = TRUE) /** * Asserts that a file exists physically on disk. * - * Overrides PHPUnit\Framework\Assert::assertFileExists() to also work with + * Overrides PHPUnit_Framework_Assert::assertFileExists() to also work with * file entities. * * @param \Drupal\File\FileInterface|string $file @@ -286,7 +286,7 @@ public function assertFileEntryExists($file, $message = NULL) { /** * Asserts that a file does not exist on disk. * - * Overrides PHPUnit\Framework\Assert::assertFileExists() to also work with + * Overrides PHPUnit_Framework_Assert::assertFileExists() to also work with * file entities. * * @param \Drupal\File\FileInterface|string $file diff --git a/core/modules/forum/src/Form/ContainerForm.php b/core/modules/forum/src/Form/ContainerForm.php index 0b01124..d12d098 100644 --- a/core/modules/forum/src/Form/ContainerForm.php +++ b/core/modules/forum/src/Form/ContainerForm.php @@ -20,8 +20,9 @@ class ContainerForm extends ForumForm { * {@inheritdoc} */ public function form(array $form, FormStateInterface $form_state) { + $taxonomy_term = $this->entity; // Build the bulk of the form from the parent forum form. - $form = parent::form($form, $form_state); + $form = parent::form($form, $form_state, $taxonomy_term); // Set the title and description of the name field. $form['name']['#title'] = $this->t('Container name'); diff --git a/core/modules/forum/src/Form/ForumForm.php b/core/modules/forum/src/Form/ForumForm.php index 9e5895d..ed0a768 100644 --- a/core/modules/forum/src/Form/ForumForm.php +++ b/core/modules/forum/src/Form/ForumForm.php @@ -29,8 +29,9 @@ class ForumForm extends TermForm { * {@inheritdoc} */ public function form(array $form, FormStateInterface $form_state) { + $taxonomy_term = $this->entity; // Build the bulk of the form from the parent taxonomy term form. - $form = parent::form($form, $form_state); + $form = parent::form($form, $form_state, $taxonomy_term); // Set the title and description of the name field. $form['name']['#title'] = $this->t('Forum name'); @@ -47,7 +48,7 @@ public function form(array $form, FormStateInterface $form_state) { // Our parent field is different to the taxonomy term. $form['parent']['#tree'] = TRUE; - $form['parent'][0] = $this->forumParentSelect($this->entity->id(), $this->t('Parent')); + $form['parent'][0] = $this->forumParentSelect($taxonomy_term->id(), $this->t('Parent')); $form['#theme_wrappers'] = ['form__forum']; $this->forumFormType = $this->t('forum'); diff --git a/core/modules/hal/tests/src/Functional/EntityResource/Editor/EditorHalJsonAnonTest.php b/core/modules/hal/tests/src/Functional/EntityResource/Editor/EditorHalJsonAnonTest.php deleted file mode 100644 index 9b68c54..0000000 --- a/core/modules/hal/tests/src/Functional/EntityResource/Editor/EditorHalJsonAnonTest.php +++ /dev/null @@ -1,30 +0,0 @@ -serializer->encode($normalization, static::$format); - // DX: 422 when incorrect entity type bundle is specified. + // DX: 400 when incorrect entity type bundle is specified. $response = $this->request($method, $url, $request_options); - $this->assertResourceErrorResponse(422, 'No entity type(s) specified', $response); + $this->assertResourceErrorResponse(400, 'No entity type(s) specified', $response); unset($normalization['_links']['type']); $request_options[RequestOptions::BODY] = $this->serializer->encode($normalization, static::$format); - // DX: 422 when no entity type bundle is specified. + // DX: 400 when no entity type bundle is specified. $response = $this->request($method, $url, $request_options); - $this->assertResourceErrorResponse(422, 'The type link relation must be specified.', $response); + $this->assertResourceErrorResponse(400, 'The type link relation must be specified.', $response); } } diff --git a/core/modules/hal/tests/src/Functional/EntityResource/Node/NodeHalJsonAnonTest.php b/core/modules/hal/tests/src/Functional/EntityResource/Node/NodeHalJsonAnonTest.php index d4ed47c..e92a3d6 100644 --- a/core/modules/hal/tests/src/Functional/EntityResource/Node/NodeHalJsonAnonTest.php +++ b/core/modules/hal/tests/src/Functional/EntityResource/Node/NodeHalJsonAnonTest.php @@ -59,20 +59,20 @@ protected function getExpectedNormalizedEntity() { 'type' => [ 'href' => $this->baseUrl . '/rest/type/node/camelids', ], - $this->baseUrl . '/rest/relation/node/camelids/revision_uid' => [ + $this->baseUrl . '/rest/relation/node/camelids/uid' => [ [ 'href' => $this->baseUrl . '/user/' . $author->id() . '?_format=hal_json', + 'lang' => 'en', ], ], - $this->baseUrl . '/rest/relation/node/camelids/uid' => [ + $this->baseUrl . '/rest/relation/node/camelids/revision_uid' => [ [ 'href' => $this->baseUrl . '/user/' . $author->id() . '?_format=hal_json', - 'lang' => 'en', ], ], ], '_embedded' => [ - $this->baseUrl . '/rest/relation/node/camelids/revision_uid' => [ + $this->baseUrl . '/rest/relation/node/camelids/uid' => [ [ '_links' => [ 'self' => [ @@ -85,9 +85,10 @@ protected function getExpectedNormalizedEntity() { 'uuid' => [ ['value' => $author->uuid()] ], + 'lang' => 'en', ], ], - $this->baseUrl . '/rest/relation/node/camelids/uid' => [ + $this->baseUrl . '/rest/relation/node/camelids/revision_uid' => [ [ '_links' => [ 'self' => [ @@ -100,7 +101,6 @@ protected function getExpectedNormalizedEntity() { 'uuid' => [ ['value' => $author->uuid()] ], - 'lang' => 'en', ], ], ], diff --git a/core/modules/hal/tests/src/Functional/EntityResource/ShortcutSet/ShortcutSetHalJsonAnonTest.php b/core/modules/hal/tests/src/Functional/EntityResource/ShortcutSet/ShortcutSetHalJsonAnonTest.php deleted file mode 100644 index a1cdf27..0000000 --- a/core/modules/hal/tests/src/Functional/EntityResource/ShortcutSet/ShortcutSetHalJsonAnonTest.php +++ /dev/null @@ -1,30 +0,0 @@ - $dimensions['width'], '#height' => $dimensions['height'], '#attributes' => $variables['attributes'], + '#uri' => $style->buildUrl($variables['uri']), '#style_name' => $variables['style_name'], ]; - // If the current image toolkit supports this file type, prepare the URI for - // the derivative image. If not, just use the original image resized to the - // dimensions specified by the style. - if ($style->supportsUri($variables['uri'])) { - $variables['image']['#uri'] = $style->buildUrl($variables['uri']); - } - else { - $variables['image']['#uri'] = $variables['uri']; - // Don't render the image by default, but allow other preprocess functions - // to override that if they need to. - $variables['image']['#access'] = FALSE; - - // Inform the site builders why their image didn't work. - \Drupal::logger('image')->warning('Could not apply @style image style to @uri because the style does not support it.', [ - '@style' => $style->label(), - '@uri' => $variables['uri'], - ]); - } - if (isset($variables['alt']) || array_key_exists('alt', $variables)) { $variables['image']['#alt'] = $variables['alt']; } diff --git a/core/modules/image/src/Entity/ImageStyle.php b/core/modules/image/src/Entity/ImageStyle.php index dd26124..fb096a7 100644 --- a/core/modules/image/src/Entity/ImageStyle.php +++ b/core/modules/image/src/Entity/ImageStyle.php @@ -14,11 +14,9 @@ use Drupal\image\ImageStyleInterface; use Drupal\Component\Utility\Crypt; use Drupal\Component\Utility\UrlHelper; -use Drupal\Component\Utility\Unicode; use Drupal\Core\StreamWrapper\StreamWrapperInterface; use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException; use Drupal\Core\Entity\Entity\EntityViewDisplay; - /** * Defines an image style configuration entity. * @@ -276,8 +274,9 @@ public function flush($path = NULL) { * {@inheritdoc} */ public function createDerivative($original_uri, $derivative_uri) { + // If the source file doesn't exist, return FALSE without creating folders. - $image = $this->getImageFactory()->get($original_uri); + $image = \Drupal::service('image.factory')->get($original_uri); if (!$image->isValid()) { return FALSE; } @@ -344,18 +343,6 @@ public function deleteImageEffect(ImageEffectInterface $effect) { /** * {@inheritdoc} */ - public function supportsUri($uri) { - // Only support the URI if its extension is supported by the current image - // toolkit. - return in_array( - Unicode::strtolower(pathinfo($uri, PATHINFO_EXTENSION)), - $this->getImageFactory()->getSupportedExtensions() - ); - } - - /** - * {@inheritdoc} - */ public function getEffect($effect) { return $this->getEffects()->get($effect); } @@ -422,16 +409,6 @@ protected function getImageEffectPluginManager() { } /** - * Returns the image factory. - * - * @return \Drupal\Core\Image\ImageFactory - * The image factory. - */ - protected function getImageFactory() { - return \Drupal::service('image.factory'); - } - - /** * Gets the Drupal private key. * * @return string diff --git a/core/modules/image/src/ImageStyleInterface.php b/core/modules/image/src/ImageStyleInterface.php index 7dec361..89fd060 100644 --- a/core/modules/image/src/ImageStyleInterface.php +++ b/core/modules/image/src/ImageStyleInterface.php @@ -194,15 +194,4 @@ public function addImageEffect(array $configuration); */ public function deleteImageEffect(ImageEffectInterface $effect); - /** - * Determines if this style can be applied to a given image. - * - * @param string $uri - * The URI of the image. - * - * @return bool - * TRUE if the image is supported, FALSE otherwise. - */ - public function supportsUri($uri); - } diff --git a/core/modules/image/tests/src/Kernel/ImageFormatterTest.php b/core/modules/image/tests/src/Kernel/ImageFormatterTest.php index e994f12..f001743 100644 --- a/core/modules/image/tests/src/Kernel/ImageFormatterTest.php +++ b/core/modules/image/tests/src/Kernel/ImageFormatterTest.php @@ -7,8 +7,6 @@ use Drupal\entity_test\Entity\EntityTest; use Drupal\field\Entity\FieldConfig; use Drupal\field\Entity\FieldStorageConfig; -use Drupal\file\Entity\File; -use Drupal\image\Entity\ImageStyle; use Drupal\Tests\field\Kernel\FieldKernelTestBase; /** @@ -101,89 +99,4 @@ public function testImageFormatterCacheTags() { $this->assertEquals($entity->{$this->fieldName}[1]->entity->getCacheTags(), $build[$this->fieldName][1]['#cache']['tags'], 'Second image cache tags is as expected'); } - /** - * Tests ImageFormatter's handling of SVG images. - * - * @requires extension gd - */ - public function testImageFormatterSvg() { - // Install the default image styles. - $this->installConfig(['image']); - - /** @var \Drupal\Core\Render\RendererInterface $renderer */ - $renderer = $this->container->get('renderer'); - - $png = File::create([ - 'uri' => 'public://test-image.png', - ]); - $png->save(); - - // We need to create an actual empty PNG, or the GD toolkit will not - // consider the image valid. - $png_resource = imagecreate(300, 300); - imagefill($png_resource, 0, 0, imagecolorallocate($png_resource, 0, 0, 0)); - imagepng($png_resource, $png->getFileUri()); - - $svg = File::create([ - 'uri' => 'public://test-image.svg', - ]); - $svg->save(); - // We don't have to put any real SVG data in here, because the GD toolkit - // won't be able to load it anyway. - touch($svg->getFileUri()); - - $entity = EntityTest::create([ - 'name' => $this->randomMachineName(), - $this->fieldName => [$png, $svg], - ]); - $entity->save(); - - // Ensure that the display is using the medium image style. - $component = $this->display->getComponent($this->fieldName); - $component['settings']['image_style'] = 'medium'; - $this->display->setComponent($this->fieldName, $component)->save(); - - $build = $this->display->build($entity); - - // The first image is a PNG, so it is supported by the GD image toolkit. - // The image style should be applied with its cache tags, image derivative - // computed with its URI and dimensions. - $this->assertCacheTags($build[$this->fieldName][0], ImageStyle::load('medium')->getCacheTags()); - $renderer->renderRoot($build[$this->fieldName][0]); - $this->assertEquals('medium', $build[$this->fieldName][0]['#image_style']); - // We check that the image URL contains the expected style directory - // structure. - $this->assertTrue(strpos($build[$this->fieldName][0]['#markup'], 'styles/medium/public/test-image.png') !== FALSE); - $this->assertTrue(strpos($build[$this->fieldName][0]['#markup'], 'width="220"') !== FALSE); - $this->assertTrue(strpos($build[$this->fieldName][0]['#markup'], 'height="220"') !== FALSE); - - // The second image is an SVG, which is not supported by the GD toolkit. - // The image style should still be applied with its cache tags, but image - // derivative will not be available so
' . t('The RESTful Web Services module provides a framework for exposing REST resources on your site. It provides support for content entity types such as the main site content, comments, custom blocks, taxonomy terms, and user accounts, etc. (see the Field module help page for more information about entities). REST support for content items of the Node module is enabled by default, and support for other types of content entities can be enabled. Other modules may add support for other types of REST resources. For more information, see the online documentation for the RESTful Web Services module.', [':rest' => 'https://www.drupal.org/documentation/modules/rest', ':field' => (\Drupal::moduleHandler()->moduleExists('field')) ? \Drupal::url('help.page', ['name' => 'field']) : '#']) . '
'; + $output .= '' . t('The RESTful Web Services module provides a framework for exposing REST resources on your site. It provides support for content entities (see the Field module help page for more information about entities) such as content, users, taxonomy terms, etc.; REST support for content items of the Node module is enabled by default, and support for other types of content entities can be enabled. Other modules may add support for other types of REST resources. For more information, see the online documentation for the RESTful Web Services module.', [':rest' => 'https://www.drupal.org/documentation/modules/rest', ':field' => (\Drupal::moduleHandler()->moduleExists('field')) ? \Drupal::url('help.page', ['name' => 'field']) : '#']) . '
'; $output .= '