bug symfony#13238 Fixed all permanently redirected and broken URLs (wouterj)

This PR was merged into the 4.4 branch.

Discussion
----------

Fixed all permanently redirected and broken URLs

I ran the linkcheck builder of the 4.4 docs and fixed all broken or permanently redirected links. Most of them are about moving to HTTPS.

We can't run the linkcheck builder automatically:

- Some docs use permanent redirects to redirect `current` to a specific version (seems really bad). We want to keep using "current" here
- In a similair way, some sites allow i18n if you omit the locale from the URL, these are also permanently redirected
- I don't want us to spam all the links every time GitHub actions runs :)

Also note that Doctrine switched the "latest" version to Doctrine ORM 3.0. I've changed all these to "current" (i.e. 2.6 atm) instead.

Commits
-------

7f8c72a Fixed all permanently redirected and broken URLs

Author: wouterj

best_practices.rst

@@ -10,7 +10,8 @@ You can even ignore them completely and continue using your own best practices
 and methodologies. Symfony is flexible enough to adapt to your needs.
 
 This article assumes that you already have experience developing Symfony
-applications. If you don't, read first the rest of the `Symfony documentation`_.
+applications. If you don't, read first the :doc:`Getting Started </setup>`
+section of the documentation.
 
 .. tip::
 
@@ -440,7 +441,6 @@ That's why it's recommended to use raw URLs in tests instead of generating them
 from routes. Whenever a route changes, tests will break and you'll know that
 you must set up a redirection.
 
-.. _`Symfony documentation`: https://symfony.com/doc
 .. _`Symfony Demo`: https://github.com/symfony/demo
 .. _`download Symfony`: https://symfony.com/download
 .. _`Composer`: https://getcomposer.org/

bundles/override.rst

@@ -180,4 +180,4 @@ For example, to override the translations defined in the
 ``Resources/translations/FOSUserBundle.es.yml`` file of the FOSUserBundle,
 create a ``<your-project>/translations/FOSUserBundle.es.yml`` file.
 
-.. _`the Doctrine documentation`: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/inheritance-mapping.html#overrides
+.. _`the Doctrine documentation`: https://www.doctrine-project.org/projects/doctrine-orm/en/current/reference/inheritance-mapping.html#overrides

components/cache.rst

@@ -201,7 +201,7 @@ Advanced Usage
 
     cache/*
 
-.. _`PSR-6`: http://www.php-fig.org/psr/psr-6/
+.. _`PSR-6`: https://www.php-fig.org/psr/psr-6/
 .. _`Cache Contracts`: https://github.com/symfony/contracts/blob/master/Cache/CacheInterface.php
 .. _`Stampede prevention`: https://en.wikipedia.org/wiki/Cache_stampede
 .. _Probabilistic early expiration: https://en.wikipedia.org/wiki/Cache_stampede#Probabilistic_early_expiration

components/cache/adapters/memcached_adapter.rst

@@ -296,8 +296,8 @@ Available Options
 .. _`User Datagram Protocol (UDP)`: https://en.wikipedia.org/wiki/User_Datagram_Protocol
 .. _`no-delay`: https://en.wikipedia.org/wiki/TCP_NODELAY
 .. _`keep-alive`: https://en.wikipedia.org/wiki/Keepalive
-.. _`Memcached PHP extension`: http://php.net/manual/en/book.memcached.php
-.. _`predefined constants`: http://php.net/manual/en/memcached.constants.php
+.. _`Memcached PHP extension`: https://www.php.net/manual/en/book.memcached.php
+.. _`predefined constants`: https://www.php.net/manual/en/memcached.constants.php
 .. _`Memcached server`: https://memcached.org/
-.. _`Memcached`: http://php.net/manual/en/class.memcached.php
+.. _`Memcached`: https://www.php.net/manual/en/class.memcached.php
 .. _`Data Source Name (DSN)`: https://en.wikipedia.org/wiki/Data_source_name

components/cache/adapters/pdo_doctrine_dbal_adapter.rst

@@ -7,7 +7,7 @@
 PDO & Doctrine DBAL Cache Adapter
 =================================
 
-This adapter stores the cache items in an SQL database. It requires a `PDO`_,
+This adapter stores the cache items in an SQL database. It requires a :phpclass:`PDO`,
 `Doctrine DBAL Connection`_, or `Data Source Name (DSN)`_ as its first parameter, and
 optionally a namespace, default cache lifetime, and options array as its second,
 third, and forth parameters::
@@ -48,6 +48,5 @@ your code.
     allowing for manual :ref:`pruning of expired cache entries <component-cache-cache-pool-prune>` by
     calling its ``prune()`` method.
 
-.. _`PDO`: http://php.net/manual/en/class.pdo.php
 .. _`Doctrine DBAL Connection`: https://github.com/doctrine/dbal/blob/master/lib/Doctrine/DBAL/Connection.php
 .. _`Data Source Name (DSN)`: https://en.wikipedia.org/wiki/Data_source_name

components/cache/adapters/php_files_adapter.rst

@@ -67,4 +67,4 @@ directory path as constructor arguments::
     allowing for manual :ref:`pruning of expired cache entries <component-cache-cache-pool-prune>` by
     calling its ``prune()`` method.
 
-.. _`OPcache`: http://php.net/manual/en/book.opcache.php
+.. _`OPcache`: https://www.php.net/manual/en/book.opcache.php

components/cache/adapters/proxy_adapter.rst

@@ -36,5 +36,5 @@ and optionally a namespace and default cache lifetime as its second and third pa
         $defaultLifetime = 0
     );
 
-.. _`PSR-6`: http://www.php-fig.org/psr/psr-6/
-.. _`cache item pool interface`: http://www.php-fig.org/psr/psr-6/#cacheitempoolinterface
+.. _`PSR-6`: https://www.php-fig.org/psr/psr-6/
+.. _`cache item pool interface`: https://www.php-fig.org/psr/psr-6/#cacheitempoolinterface

components/cache/psr6_psr16_adapters.rst

@@ -91,4 +91,4 @@ this use-case::
 
     The ``Psr16Cache`` class was introduced in Symfony 4.3.
 
-.. _`PSR-16`: http://www.php-fig.org/psr/psr-16/
+.. _`PSR-16`: https://www.php-fig.org/psr/psr-16/

components/dependency_injection.rst

@@ -318,4 +318,4 @@ Learn More
     /components/dependency_injection/*
     /service_container/*
 
-.. _`PSR-11`: http://www.php-fig.org/psr/psr-11/
+.. _`PSR-11`: https://www.php-fig.org/psr/psr-11/

components/dotenv.rst

@@ -170,4 +170,4 @@ Embed commands via ``$()`` (not supported on Windows):
 
     Note that using ``$()`` might not work depending on your shell.
 
-.. _twelve-factor applications: http://www.12factor.net/
+.. _twelve-factor applications: https://12factor.net/

components/event_dispatcher.rst

@@ -531,5 +531,5 @@ Learn More
 
 .. _Mediator: https://en.wikipedia.org/wiki/Mediator_pattern
 .. _Observer: https://en.wikipedia.org/wiki/Observer_pattern
-.. _Closures: https://php.net/manual/en/functions.anonymous.php
-.. _PHP callable: https://php.net/manual/en/language.pseudo-types.php#language.types.callback
+.. _Closures: https://www.php.net/manual/en/functions.anonymous.php
+.. _PHP callable: https://www.php.net/manual/en/language.pseudo-types.php#language.types.callback

components/finder.rst

@@ -407,7 +407,7 @@ The contents of returned files can be read with
 .. _`fluent interface`: https://en.wikipedia.org/wiki/Fluent_interface
 .. _`symbolic links`: https://en.wikipedia.org/wiki/Symbolic_link
 .. _`Version Control Systems`: https://en.wikipedia.org/wiki/Version_control
-.. _`PHP wrapper for URL-style protocols`: https://php.net/manual/en/wrappers.php
-.. _`PHP streams`: https://php.net/streams
+.. _`PHP wrapper for URL-style protocols`: https://www.php.net/manual/en/wrappers.php
+.. _`PHP streams`: https://www.php.net/streams
 .. _`IEC standard`: https://physics.nist.gov/cuu/Units/binary.html
 .. _`natural sort order`: https://en.wikipedia.org/wiki/Natural_sort_order

components/http_client.rst

@@ -1032,7 +1032,7 @@ However, using ``MockResponse`` allows simulating chunked responses and timeouts
 
     $mockResponse = new MockResponse($body());
 
-.. _`cURL PHP extension`: https://php.net/curl
+.. _`cURL PHP extension`: https://www.php.net/curl
 .. _`PSR-17`: https://www.php-fig.org/psr/psr-17/
 .. _`PSR-18`: https://www.php-fig.org/psr/psr-18/
 .. _`HTTPlug`: https://github.com/php-http/httplug/#readme

components/http_foundation.rst

@@ -725,5 +725,5 @@ Learn More
 
 .. _nginx: https://www.nginx.com/resources/wiki/start/topics/examples/xsendfile/
 .. _Apache: https://tn123.org/mod_xsendfile/
-.. _`JSON Hijacking`: http://haacked.com/archive/2009/06/25/json-hijacking.aspx
+.. _`JSON Hijacking`: https://haacked.com/archive/2009/06/25/json-hijacking.aspx/
 .. _OWASP guidelines: https://cheatsheetseries.owasp.org/cheatsheets/AJAX_Security_Cheat_Sheet.html#always-return-json-with-an-object-on-the-outside

components/http_foundation/session_configuration.rst

@@ -29,7 +29,7 @@ All native save handlers are internal to PHP and as such, have no public facing
 They must be configured by ``php.ini`` directives, usually ``session.save_path`` and
 potentially other driver specific directives. Specific details can be found in
 the docblock of the ``setOptions()`` method of each class. For instance, the one
-provided by the Memcached extension can be found on `php.net/memcached.setoption`_
+provided by the Memcached extension can be found on :phpmethod:`php.net <Memcached::setOption>`.
 
 While native save handlers can be activated by directly using
 ``ini_set('session.save_handler', $name);``, Symfony provides a convenient way to
@@ -286,6 +286,5 @@ particular cookie by reading the ``getLifetime()`` method::
 The expiry time of the cookie can be determined by adding the created
 timestamp and the lifetime.
 
-.. _`php.net/session.customhandler`: https://php.net/session.customhandler
-.. _`php.net/session.configuration`: https://php.net/session.configuration
-.. _`php.net/memcached.setoption`: https://php.net/memcached.setoption
+.. _`php.net/session.customhandler`: https://www.php.net/session.customhandler
+.. _`php.net/session.configuration`: https://www.php.net/session.configuration

components/http_kernel.rst

@@ -777,11 +777,11 @@ Learn more
 
    /reference/events
 
-.. _reflection: https://php.net/manual/en/book.reflection.php
+.. _reflection: https://www.php.net/manual/en/book.reflection.php
 .. _FOSRestBundle: https://github.com/friendsofsymfony/FOSRestBundle
-.. _`PHP FPM`: https://php.net/manual/en/install.fpm.php
+.. _`PHP FPM`: https://www.php.net/manual/en/install.fpm.php
 .. _`SensioFrameworkExtraBundle`: https://symfony.com/doc/current/bundles/SensioFrameworkExtraBundle/index.html
 .. _`@ParamConverter`: https://symfony.com/doc/current/bundles/SensioFrameworkExtraBundle/annotations/converters.html
 .. _`@Template`: https://symfony.com/doc/current/bundles/SensioFrameworkExtraBundle/annotations/view.html
 .. _`EmailSenderListener`: https://github.com/symfony/swiftmailer-bundle/blob/master/EventListener/EmailSenderListener.php
-.. _variadic: http://php.net/manual/en/functions.arguments.php
+.. _variadic: https://www.php.net/manual/en/functions.arguments.php#functions.variable-arg-list

components/intl.rst

@@ -406,8 +406,8 @@ Learn more
     /reference/forms/types/locale
     /reference/forms/types/timezone
 
-.. _intl extension: https://php.net/manual/en/book.intl.php
-.. _install the intl extension: https://php.net/manual/en/intl.setup.php
+.. _intl extension: https://www.php.net/manual/en/book.intl.php
+.. _install the intl extension: https://www.php.net/manual/en/intl.setup.php
 .. _ICU library: http://site.icu-project.org/
 .. _`Unicode ISO 15924 Registry`: https://www.unicode.org/iso15924/iso15924-codes.html
 .. _`ISO 3166-1 alpha-2`: https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2

components/lock.rst

@@ -685,8 +685,8 @@ are still running.
 
 .. _`ACID`: https://en.wikipedia.org/wiki/ACID
 .. _`locks`: https://en.wikipedia.org/wiki/Lock_(computer_science)
-.. _`PHP semaphore functions`: https://php.net/manual/en/book.sem.php
-.. _`PDO`: https://php.net/pdo
+.. _`PHP semaphore functions`: https://www.php.net/manual/en/book.sem.php
+.. _`PDO`: https://www.php.net/pdo
 .. _`Doctrine DBAL Connection`: https://github.com/doctrine/dbal/blob/master/lib/Doctrine/DBAL/Connection.php
 .. _`Data Source Name (DSN)`: https://en.wikipedia.org/wiki/Data_source_name
 .. _`ZooKeeper`: https://zookeeper.apache.org/

components/messenger.rst

@@ -124,6 +124,8 @@ that will do the required processing for your message::
         }
     }
 
+.. _messenger-envelopes:
+
 Adding Metadata to Messages (Envelopes)
 ---------------------------------------
 

components/mime.rst

@@ -302,4 +302,4 @@ You can register your own MIME type guesser by creating a class that implements
 
 .. _`MIME`: https://en.wikipedia.org/wiki/MIME
 .. _`MIME types`: https://en.wikipedia.org/wiki/Media_type
-.. _`fileinfo extension`: https://php.net/fileinfo
+.. _`fileinfo extension`: https://www.php.net/fileinfo

components/phpunit_bridge.rst

@@ -223,9 +223,9 @@ message contains the ``"foobar"`` string.
 Making Tests Fail
 ~~~~~~~~~~~~~~~~~
 
-By default, any non-legacy-tagged or any non-`@-silenced`_ deprecation
-notices will make tests fail. Alternatively, you can configure an
-arbitrary threshold by setting ``SYMFONY_DEPRECATIONS_HELPER`` to
+By default, any non-legacy-tagged or any non-`@-silenced <@-silencing operator>`_
+deprecation notices will make tests fail. Alternatively, you can configure
+an arbitrary threshold by setting ``SYMFONY_DEPRECATIONS_HELPER`` to
 ``max[total]=320`` for instance. It will make the tests fails only if a
 higher number of deprecation notices is reached (``0`` is the default
 value).
@@ -971,11 +971,10 @@ not find the SUT:
 .. _`PHPUnit`: https://phpunit.de
 .. _`PHPUnit event listener`: https://phpunit.de/manual/current/en/extending-phpunit.html#extending-phpunit.PHPUnit_Framework_TestListener
 .. _`PHPUnit's assertStringMatchesFormat()`: https://phpunit.de/manual/current/en/appendixes.assertions.html#appendixes.assertions.assertStringMatchesFormat
-.. _`PHP error handler`: https://php.net/manual/en/book.errorfunc.php
+.. _`PHP error handler`: https://www.php.net/manual/en/book.errorfunc.php
 .. _`environment variable`: https://phpunit.de/manual/current/en/appendixes.configuration.html#appendixes.configuration.php-ini-constants-variables
-.. _`@-silencing operator`: https://php.net/manual/en/language.operators.errorcontrol.php
-.. _`@-silenced`: https://php.net/manual/en/language.operators.errorcontrol.php
+.. _`@-silencing operator`: https://www.php.net/manual/en/language.operators.errorcontrol.php
 .. _`Travis CI`: https://travis-ci.org/
 .. _`test listener`: https://phpunit.de/manual/current/en/appendixes.configuration.html#appendixes.configuration.test-listeners
 .. _`@covers`: https://phpunit.de/manual/current/en/appendixes.annotations.html#appendixes.annotations.covers
-.. _`PHP namespace resolutions rules`: https://php.net/manual/en/language.namespaces.rules.php
+.. _`PHP namespace resolutions rules`: https://www.php.net/manual/en/language.namespaces.rules.php

components/serializer.rst

@@ -1507,7 +1507,7 @@ Learn more
 .. _RFC3339: https://tools.ietf.org/html/rfc3339#section-5.8
 .. _JSON: http://www.json.org/
 .. _XML: https://www.w3.org/XML/
-.. _YAML: http://yaml.org/
+.. _YAML: https://yaml.org/
 .. _CSV: https://tools.ietf.org/html/rfc4180
 .. _`RFC 7807`: https://tools.ietf.org/html/rfc7807
 .. _`Value Objects`: https://en.wikipedia.org/wiki/Value_object

components/validator.rst

@@ -88,4 +88,4 @@ Learn More
     /validation
     /validation/*
 
-.. _`JSR-303 Bean Validation specification`: http://jcp.org/en/jsr/detail?id=303
+.. _`JSR-303 Bean Validation specification`: https://jcp.org/en/jsr/detail?id=303

components/var_exporter.rst

@@ -128,5 +128,5 @@ created by using the special ``"\0"`` property name to define their internal val
         "\0" => [$inputArray]
     ]);
 
-.. _`OPcache`: https://php.net/opcache
+.. _`OPcache`: https://www.php.net/opcache
 .. _`PSR-2`: https://www.php-fig.org/psr/psr-2/

components/yaml.rst

@@ -459,6 +459,6 @@ Learn More
 
     yaml/*
 
-.. _`YAML`: http://yaml.org/
-.. _`YAML 1.2 version specification`: http://yaml.org/spec/1.2/spec.html
-.. _`ISO-8601`: http://www.iso.org/iso/iso8601
+.. _`YAML`: https://yaml.org/
+.. _`YAML 1.2 version specification`: https://yaml.org/spec/1.2/spec.html
+.. _`ISO-8601`: https://www.iso.org/iso-8601-date-and-time-format.html

components/yaml/yaml_format.rst

@@ -333,6 +333,6 @@ The following YAML features are not supported by the Symfony Yaml component:
 * Using sequence-like syntax for mapping elements (example: ``{foo, bar}``; use
   ``{foo: ~, bar: ~}`` instead).
 
-.. _`ISO-8601`: http://www.iso.org/iso/iso8601
-.. _`YAML website`: http://yaml.org/
-.. _`YAML specification`: http://www.yaml.org/spec/1.2/spec.html
+.. _`ISO-8601`: https://www.iso.org/iso-8601-date-and-time-format.html
+.. _`YAML website`: https://yaml.org/
+.. _`YAML specification`: https://www.yaml.org/spec/1.2/spec.html

configuration/micro_kernel_trait.rst

@@ -80,9 +80,7 @@ That's it! To test it, start the :doc:`Symfony Local Web Server
 
     $ symfony server:start
 
-Then see the JSON response in your browser:
-
-    http://localhost:8000/random/10
+Then see the JSON response in your browser: http://localhost:8000/random/10
 
 The Methods of a "Micro" Kernel
 -------------------------------
@@ -332,6 +330,4 @@ As before you can use the :doc:`Symfony Local Web Server
     cd public/
     $ symfony server:start
 
-Then visit the page in your browser:
-
-    http://localhost:8000/random/10
+Then visit the page in your browser: http://localhost:8000/random/10

contributing/code/conventions.rst

@@ -125,8 +125,6 @@ notices. Silencing swaps this behavior and allows users to opt-in when they are
 ready to cope with them (by adding a custom error handler like the one used by
 the Web Debug Toolbar or by the PHPUnit bridge).
 
-.. _`@-silencing operator`: https://php.net/manual/en/language.operators.errorcontrol.php
-
 When deprecating a whole class the ``trigger_error()`` call should be placed
 between the namespace and the use declarations, like in this example from
 `ServiceRouterLoader`_::
@@ -184,3 +182,5 @@ of the impacted component::
     * Removed the `Deprecated` class, use `Replacement` instead.
 
 This task is mandatory and must be done in the same pull request.
+
+.. _`@-silencing operator`: https://www.php.net/manual/en/language.operators.errorcontrol.php

contributing/code/pull_requests.rst

@@ -415,7 +415,7 @@ before merging.
 
 .. _ProGit: https://git-scm.com/book
 .. _GitHub: https://github.com/join
-.. _`GitHub's documentation`: https://help.github.com/articles/ignoring-files
+.. _`GitHub's documentation`: https://help.github.com/github/using-git/ignoring-files
 .. _Symfony repository: https://github.com/symfony/symfony
 .. _`documentation repository`: https://github.com/symfony/symfony-docs
 .. _`fabbot`: https://fabbot.io
@@ -424,4 +424,4 @@ before merging.
 .. _`searching on GitHub`: https://github.com/symfony/symfony/issues?q=+is%3Aopen+
 .. _`Symfony Slack`: https://symfony.com/slack-invite
 .. _`Travis-CI`: https://travis-ci.org/symfony/symfony
-.. _`draft status`: https://help.github.com/en/articles/about-pull-requests#draft-pull-requests
+.. _`draft status`: https://help.github.com/github/collaborating-with-issues-and-pull-requests/about-pull-requests#draft-pull-requests

contributing/code/standards.rst

@@ -281,7 +281,7 @@ License
 .. _`PSR-1`: https://www.php-fig.org/psr/psr-1/
 .. _`PSR-2`: https://www.php-fig.org/psr/psr-2/
 .. _`PSR-4`: https://www.php-fig.org/psr/psr-4/
-.. _`identical comparison`: https://php.net/manual/en/language.operators.comparison.php
+.. _`identical comparison`: https://www.php.net/manual/en/language.operators.comparison.php
 .. _`Yoda conditions`: https://en.wikipedia.org/wiki/Yoda_conditions
 .. _`camelCase`: https://en.wikipedia.org/wiki/Camel_case
 .. _`UpperCamelCase`: https://en.wikipedia.org/wiki/Camel_case

contributing/code/tests.rst

@@ -55,7 +55,7 @@ what's going on and if the tests are broken because of the new code.
     to see colored test results.
 
 .. _`install Composer`: https://getcomposer.org/download/
-.. _Cmder: http://cmder.net/
+.. _Cmder: https://cmder.net/
 .. _ConEmu: https://conemu.github.io/
 .. _ANSICON: https://github.com/adoxa/ansicon/releases
 .. _Mintty: https://mintty.github.io/

contributing/code_of_conduct/code_of_conduct.rst

@@ -77,7 +77,7 @@ Attribution
 -----------
 
 This Code of Conduct is adapted from the `Contributor Covenant`_, version 1.4,
-available at https://www.contributor-covenant.org/version/1/4/code-of-conduct.html
+available at https://www.contributor-covenant.org/version/1/4/code-of-conduct/
 
 Related Documents
 -----------------

contributing/community/releases.rst

@@ -132,6 +132,6 @@ period to upgrade. Companies wanting more stability use the LTS versions: a new
 version is published every two years and there is a year to upgrade.
 
 .. _`semantic versioning`: https://semver.org/
-.. _`Subscribe to Symfony Roadmap notifications`: https://symfony.com/account
-.. _`Symfony Roadmap`: https://symfony.com/roadmap#checker
+.. _`Subscribe to Symfony Roadmap notifications`: https://symfony.com/account/notifications
+.. _`Symfony Roadmap`: https://symfony.com/releases
 .. _`professional Symfony support`: https://sensiolabs.com/

contributing/community/reviews.rst

@@ -213,9 +213,9 @@ Pick a pull request from the `PRs in need of review`_ and follow these steps:
 .. _Symfony issue tracker: https://github.com/symfony/symfony/issues
 .. _`Symfony skeleton`: https://github.com/symfony/skeleton
 .. _`Symfony website skeleton`: https://github.com/symfony/website-skeleton
-.. _create a GitHub account: https://help.github.com/articles/signing-up-for-a-new-github-account/
+.. _create a GitHub account: https://help.github.com/github/getting-started-with-github/signing-up-for-a-new-github-account
 .. _bug reports in need of review: https://github.com/symfony/symfony/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Aissue+label%3A%22Bug%22+label%3A%22Status%3A+Needs+Review%22+
-.. _PRs in need of review: https://github.com/symfony/symfony/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Apr+label%3A%22Status%3A+Needs+Review%22+
-.. _Symfony Roadmap: https://symfony.com/roadmap
+.. _PRs in need of review: https://github.com/symfony/symfony/pulls?q=is%3Aopen+is%3Apr+label%3A%22Status%3A+Needs+Review%22
+.. _Symfony Roadmap: https://symfony.com/releases
 .. _Carson Bot: https://github.com/carsonbot/carsonbot
 .. _`Needs Review`: https://github.com/symfony/symfony/labels/Status%3A%20Needs%20Review