From d4c162752613a3cccb5c70ad6fa5d066aafa3b01 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Edi=20Modri=C4=87?= <edi@netgen.hr>
Date: Tue, 27 Jun 2017 13:55:18 +0200
Subject: [PATCH] Add upgrade instructions for version 0.8.0

---
 index.rst                    |  10 +++
 upgrades/index.rst           |   9 +++
 upgrades/map.rst.inc         |   1 +
 upgrades/upgrade_070_080.rst | 148 +++++++++++++++++++++++++++++++++++
 4 files changed, 168 insertions(+)
 create mode 100644 upgrades/index.rst
 create mode 100644 upgrades/map.rst.inc
 create mode 100644 upgrades/upgrade_070_080.rst

diff --git a/index.rst b/index.rst
index ee46e81..5eba2f1 100644
--- a/index.rst
+++ b/index.rst
@@ -32,6 +32,16 @@ Reference
 
 .. include:: /reference/map.rst.inc
 
+Upgrades
+--------
+
+.. toctree::
+    :hidden:
+
+    upgrades/index
+
+.. include:: /upgrades/map.rst.inc
+
 The cookbook
 ------------
 
diff --git a/upgrades/index.rst b/upgrades/index.rst
new file mode 100644
index 0000000..c3019af
--- /dev/null
+++ b/upgrades/index.rst
@@ -0,0 +1,9 @@
+Upgrades
+========
+
+.. toctree::
+    :hidden:
+
+    upgrade_070_080
+
+.. include:: /upgrades/map.rst.inc
diff --git a/upgrades/map.rst.inc b/upgrades/map.rst.inc
new file mode 100644
index 0000000..709f8e9
--- /dev/null
+++ b/upgrades/map.rst.inc
@@ -0,0 +1 @@
+* :doc:`/upgrades/upgrade_070_080`
diff --git a/upgrades/upgrade_070_080.rst b/upgrades/upgrade_070_080.rst
new file mode 100644
index 0000000..5083bf1
--- /dev/null
+++ b/upgrades/upgrade_070_080.rst
@@ -0,0 +1,148 @@
+Upgrading from 0.7.0 to 0.8.0
+=============================
+
+Upgrade ``composer.json``
+~~~~~~~~~~~~~~~~~~~~~~~~~
+
+In your ``composer.json`` file, upgrade the version of ``netgen/content-browser``
+package to ``~0.8.0`` and run the ``composer update`` command.
+
+Breaking changes
+~~~~~~~~~~~~~~~~
+
+The following breaking changes were made in version ``0.8`` of Content Browser.
+Follow the instructions to upgrade your code to this newer version.
+
+* ``sections`` config has been removed from configuration of item types since
+  overriding sections has never been properly implemented. Remove the section
+  from your own item type definitions.
+
+* The concept of item serializer handlers is removed. Previously, they were used
+  to specify how certain data will be serialized for an item. One method that
+  the ``Netgen\ContentBrowser\Item\Serializer\ItemSerializerHandlerInterface``
+  provided has been moved to ``Netgen\ContentBrowser\Item\ItemInterface``, so
+  you need to move your implementation there:
+
+  .. code-block:: php
+
+      // Before
+
+      <?php
+
+      namespace AppBundle\ContentBrowser\Item\Serializer\Handler;
+
+      use Netgen\ContentBrowser\Item\ItemInterface;
+      use Netgen\ContentBrowser\Item\Serializer\ItemSerializerHandlerInterface;
+
+      class MyItemSerializerHandler implements ItemSerializerHandlerInterface
+      {
+          /**
+           * Returns if the item is selectable.
+           *
+           * @param \Netgen\ContentBrowser\Item\ItemInterface $item
+           *
+           * @return bool
+           */
+          public function isSelectable(ItemInterface $item)
+          {
+              // Determine if the item is selectable
+          }
+      }
+
+  .. code-block:: php
+
+      // After
+
+      <?php
+
+      namespace AppBundle\ContentBrowser\Item\MyItem;
+
+      use Netgen\ContentBrowser\Item\ItemInterface;
+
+      class Item implements ItemInterface
+      {
+          /**
+           * Returns if the item is selectable.
+           *
+           * @return bool
+           */
+          public function isSelectable()
+          {
+              // Determine if the item is selectable
+          }
+      }
+
+* The concept of item template value providers is removed. Previously, they were
+  used to provide the variables that will be used by the templates to render the
+  item preview and columns. Now, one single variable called ``item`` is injected
+  to all Twig templates. This variable holds the actual item which is rendered,
+  which means that your ``Netgen\ContentBrowser\Item\ItemInterface`` objects
+  need to provide everything that will be used to render the templates.
+
+  .. code-block:: php
+
+      // Before
+
+      <?php
+
+      namespace AppBundle\ContentBrowser\Item\Renderer\TemplateValueProvider;
+
+      use Netgen\ContentBrowser\Item\ItemInterface;
+      use Netgen\ContentBrowser\Item\Renderer\TemplateValueProviderInterface;
+
+      class MyItemTemplateValueProvider implements TemplateValueProviderInterface
+      {
+          /**
+           * Provides the values for template rendering.
+           *
+           * @param \Netgen\ContentBrowser\Item\ItemInterface $item
+           *
+           * @return array
+           */
+          public function getValues(ItemInterface $item)
+          {
+              $thingOne = ...;
+              $thingTwo = ...;
+
+              return array(
+                  'thingOne' => $thingOne,
+                  'thingTwo' => $thingTwo,
+              );
+          }
+      }
+
+  .. code-block:: php
+
+      // After
+
+      <?php
+
+      namespace AppBundle\ContentBrowser\Item\MyItem;
+
+      use Netgen\ContentBrowser\Item\ItemInterface;
+
+      class Item implements ItemInterface
+      {
+          /**
+           * Returns thing one.
+           *
+           * @return mixed
+           */
+          public function getThingOne()
+          {
+              return ...;
+          }
+
+          /**
+           * Returns thing two.
+           *
+           * @return mixed
+           */
+          public function getThingTwo()
+          {
+              return ...;
+          }
+      }
+
+  In your templates, instead of directly using variables ``thingOne`` and
+  ``thingTwo``, you will now use ``item.thingOne`` and ``item.thingTwo``.