Merge pull request #61 from City-of-Helsinki/UHF-8198

UHF-8198: Comment fixes

README.md

@@ -119,9 +119,9 @@ if ($varnish_port = getenv('DRUPAL_VARNISH_PORT')) {
   $config['varnish_purger.settings.varnish_purge_all']['port'] = $varnish_port;
 }
 
-// Configuration doesn't know about existing config yet so we can't
+// Configuration doesn't know about existing config yet, so we can't
 // just append new headers to an already existing headers array here.
-// If you have configured any extra headers in your purge settings
+// If you have configured any extra headers in your purge settings,
 // you must add them here as well.
 $config['varnish_purger.settings.default']['headers'] = [
   [

src/HelfiProxyServiceProvider.php

@@ -18,9 +18,11 @@ class HelfiProxyServiceProvider extends ServiceProviderBase {
    * {@inheritdoc}
    */
   public function alter(ContainerBuilder $container) : void {
-    // PathProcessorLanguage and SitemapPathProcessor have same
-    // priority. Alter SitemapPathProcessor's priority so
-    // SitePrefixPathProcessor can be run in between the two.
+    // PathProcessorLanguage and SitemapPathProcessor have the same
+    // priority. Alter SitemapPathProcessor's priority, so
+    // SitePrefixPathProcessor can be run in between the two. Otherwise,
+    // the routes provided by 'simple_sitemap' path won't be recognized
+    // via the proxy path.
     if ($container->hasDefinition('simple_sitemap.path_processor')) {
       $definition = $container->getDefinition('simple_sitemap.path_processor');
       $tags = $definition->getTags();

src/PathProcessor/SitePrefixPathProcessor.php

@@ -59,7 +59,7 @@ public function processOutbound(
     if ($options['language'] instanceof LanguageInterface) {
       $language = $options['language']->getId();
     }
-    // Use already resolved language to figure out active prefix
+    // Use an already resolved language to figure out active prefix
     // since it might be different from content language.
     $prefix = $this->sitePrefix->getPrefix($language);