ENH Simplify APIs

_config.php

@@ -5,5 +5,4 @@
 
 // Avoid creating global variables
 call_user_func(function () {
-
 });

client/src/boot/index.js

@@ -1,14 +1,9 @@
 /* global document */
 /* eslint-disable */
-import Config from 'lib/Config';
 import registerReducers from './registerReducers';
 import registerComponents from './registerComponents';
-import registerQueries from './registerQueries';
 
 document.addEventListener('DOMContentLoaded', () => {
   registerComponents();
-
-  registerQueries();
-
   registerReducers();
 });

client/src/components/LinkField/LinkField.js

@@ -1,37 +1,62 @@
-import React, { Fragment, useState } from 'react';
+import React, { Fragment, useState, useEffect } from 'react';
 import { compose } from 'redux';
-import { inject, injectGraphql, loadComponent } from 'lib/Injector';
+import { inject, loadComponent } from 'lib/Injector';
 import fieldHolder from 'components/FieldHolder/FieldHolder';
 
-const LinkField = ({ id, loading, Loading, data, LinkPicker, onChange, types, linkDescription, ...props }) => {
-  if (loading) {
-    return <Loading />;
-  }
-
+// id, onChange, dataStr - come from entwine/JsonField.js
+// LinkPicker, Loading - come from inject(['LinkPicker', 'Loading']),
+const LinkField = ({ id, onChange, dataStr, Loading, LinkPicker }) => {
+  const [types, setTypes] = useState([]);
+  const [description, setDescription] = useState(null);
   const [editing, setEditing] = useState(false);
   const [newTypeKey, setNewTypeKey] = useState('');
 
   const onClear = (event) => {
     if (typeof onChange !== 'function') {
       return;
     }
-
     onChange(event, { id, value: {} });
   };
 
+  // Utility function to make a GET request to the server
+  const fetchData = (path, onFetched) => {
+    (async () => fetch(path))()
+      .then(response => response.json())
+      .then(onFetched);
+  };
+
+  // Request of types data from server only on the initial load of the component
+  useEffect(() => {
+    fetchData('/admin/linkfield/types', (responseJson) => setTypes(responseJson));
+  }, []);
+
+  // Make a request to the server for the description of the link when editing changes to false
+  // and on the initial load of the component
+  useEffect(() => {
+    if (!editing) {
+      const path = `/admin/linkfield/description?data=${encodeURI(dataStr)}`;
+      fetchData(path, (responseJson) => setDescription(responseJson.description));
+    }
+  }, [editing]);
+
+  if (types.length === 0 || description === null) {
+    return <Loading />;
+  }
+
+  const data = JSON.parse(dataStr);
   const { typeKey } = data;
   const type = types[typeKey];
   const modalType = newTypeKey ? types[newTypeKey] : type;
 
   let title = data ? data.Title : '';
-
   if (!title) {
     title = data ? data.TitleRelField : '';
   }
 
   const linkProps = {
     title,
-    link: type ? { type, title, description: linkDescription } : undefined,
+    // link: type ? { type, title, description: linkDescription } : undefined,
+    link: type ? { type, title, description } : undefined,
     onEdit: () => { setEditing(true); },
     onClear,
     onSelect: (key) => {
@@ -43,14 +68,11 @@ const LinkField = ({ id, loading, Loading, data, LinkPicker, onChange, types, li
 
   const onModalSubmit = (modalData, action, submitFn) => {
     const { SecurityID, action_insert: actionInsert, ...value } = modalData;
-
     if (typeof onChange === 'function') {
       onChange(event, { id, value });
     }
-
     setEditing(false);
     setNewTypeKey('');
-
     return Promise.resolve();
   };
 
@@ -83,8 +105,6 @@ const stringifyData = (Component) => (({ data, value, ...props }) => {
 
 export default compose(
   inject(['LinkPicker', 'Loading']),
-  injectGraphql('readLinkTypes'),
   stringifyData,
-  injectGraphql('readLinkDescription'),
   fieldHolder
 )(LinkField);

src/Extensions/LeftAndMain.php

@@ -2,14 +2,21 @@
 
 namespace SilverStripe\LinkField\Extensions;
 
+use SilverStripe\Control\HTTPResponse;
 use SilverStripe\Core\Extension;
 use SilverStripe\LinkField\Type\Registry;
+use SilverStripe\Control\HTTPResponse_Exception;
+use SilverStripe\Control\HTTPRequest;
 
 /**
  * Register a new Form Schema in LeftAndMain.
  */
 class LeftAndMain extends Extension
 {
+    private static $allowed_actions = [
+        'linkfield',
+    ];
+
     public function init()
     {
         // Get the Link Registry to load all the JS requirements for managing Links.
@@ -22,4 +29,76 @@ public function updateClientConfig(&$clientConfig)
             'schemaUrl' => $this->getOwner()->Link('methodSchema/Modals/DynamicLink'),
         ];
     }
+
+    /**
+     * Endoint for /admin/linkfield/*
+     */
+    public function linkfield(): HTTPResponse
+    {
+        $request = $this->getOwner()->getRequest();
+        if (!$request->isGET()) {
+            throw new HTTPResponse_Exception('Only GET requests are allowed');
+        }
+        $action = $request->param('ID');
+        if ($action === 'description') {
+            return $this->description();
+        } elseif ($action === 'types') {
+            return $this->types();
+        }
+        throw new HTTPResponse_Exception('Invalid action');
+    }
+
+    /**
+     * /admin/linkfield/description/{json}
+     */
+    private function description(): HTTPResponse
+    {
+        /** @var HTTPRequest $request */
+        $request = $this->getOwner()->getRequest();
+        $jsonStr = $request->getVar('data');
+        // data will be an empty array if there is no existing link
+        $data = json_decode($jsonStr, true);
+        $description = '';
+        if (json_last_error() !== JSON_ERROR_NONE) {
+            throw new HTTPResponse_Exception('data is not a valid JSON string');
+        }
+        if (array_key_exists('typeKey', $data)) {
+            $typeKey = $data['typeKey'];
+            /** @var Type $type */
+            $type = Registry::singleton()->byKey($typeKey);
+            if (!$type) {
+                throw new HTTPResponse_Exception('typeKey is not allowed');
+            }
+            $description = $type->generateLinkDescription($data);
+        }
+        return $this->jsonResponse([
+            'description' => $description
+        ]);
+    }
+
+    /**
+     * /admin/linkfield/types
+     */
+    private function types(): HTTPResponse
+    {
+        $data = [];
+        /** @var Type $type */
+        foreach (Registry::singleton()->list() as $key => $type) {
+            $data[$key] = [
+                'key' => $key,
+                'handlerName' => $type->LinkTypeHandlerName(),
+                'title' => $type->LinkTypeTile()
+            ];
+        }
+        return $this->jsonResponse($data);
+    }
+
+    private function jsonResponse(array $data)
+    {
+        $response = $this->getOwner()->getResponse();
+        $jsonStr = json_encode($data, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+        $response->setBody($jsonStr);
+        $response->addHeader('Content-type', 'application/json');
+        return $response;
+    }
 }