Add support for template name change tracking.

Author: dadish

ProcessGraphQL.module

@@ -1,7 +1,9 @@
 <?php namespace ProcessWire;
 
 use Youshido\GraphQL\Execution\Processor;
+use ProcessWire\HookEvent;
 use ProcessWire\GraphQL\Schema;
+use Processwire\ProcessGraphQLConfig;
 
 class ProcessGraphQL extends Process implements Module {
 
@@ -15,7 +17,7 @@ class ProcessGraphQL extends Process implements Module {
 			'summary' => 'GraphQL for ProcessWire.',
 			'href' => 'https://github.com/dadish/ProcessGraphql',
 			'singular' => true, 
-			'autoload' => false, 
+			'autoload' => 'process=ProcessTemplate', 
 			'icon' => 'object-group',
 			);
 	}
@@ -25,9 +27,39 @@ class ProcessGraphQL extends Process implements Module {
 	 */
 	public function init()
 	{
+		$this->addHookAfter('Template::changed', $this, 'hookTemplateNameChange');
 		require_once $this->config->paths->site . 'modules/ProcessGraphQL/vendor/autoload.php';
 	}
 
+	public function hookTemplateNameChange(HookEvent $event)
+	{
+		// require_once $this->config->paths->site . 'modules/ProcessGraphQL/ProcessGraphQLConfig.php';
+		$whatChanged = $event->arguments[0];
+		$oldName = $event->arguments[1];
+		$newName = $event->arguments[2];
+
+		// do nothing if name has not changed
+		if ($whatChanged !== 'name') return;
+		
+		// do nothing if template is not in legalTemplates
+		if (!in_array($oldName, $this->legalTemplates)) return;
+		
+		// remove the oldName from the legalTemplates
+		$index = array_search($oldName, $this->legalTemplates);
+		$legalTemplates = array_slice($this->legalTemplates, 0);
+		array_splice($legalTemplates, $index, 1);
+		$this->message("Removed `$oldName` from the legalTemplates.");
+
+		// if newName is compatible with the module then add it into legalTemplates
+		if (ProcessGraphQLConfig::isLegalTemplateName($newName)) {
+			$legalTemplates[] = $newName;
+			$this->message("Added `$newName` into legalTemplaes.");
+		}
+
+		// make sure to remember changes!
+		$this->modules->saveConfig($this, 'legalTemplates', $legalTemplates);
+	}
+
   public function ___execute()
   {
     if ($this->config->ajax) return $this->executeGraphQL();

ProcessGraphQLConfig.php

@@ -28,7 +28,7 @@ public function getDefaults()
         'httpUrl',
         'description',
       ],
-      'fullWidthGraphiql' => false,
+      'fullWidthGraphiQL' => false,
     );
   }
 
@@ -37,6 +37,7 @@ public static function isLegalTemplateName($name)
 		if (!$name) return false;
 		if (preg_match('/^[_A-Za-z][-_0-9A-Za-z]*$/', $name) !== 1) return false; // the GraphQL naming requirement
 		if (strpos($name, '__') === 0) return false; // the names with `__` prefix are reserved by GraphQL
+    if (strpos($name, '--') === 0) return false; // as we change the `-` symbols to `_` for field names the `--` becomes `__` and it also reserved by GraphQL
 		if (in_array($name, Settings::getReservedWords())) return false; // some words that used now and might be for future
     return true;
 	}