[WIP] Add consistent Layout + interface

lib/Fhaculty/Graph/Algorithm/MaximumMatching/Flow.php

@@ -30,8 +30,8 @@ public function getEdges()
         // create temporary flow graph with supersource and supersink
         $graphFlow = $this->graph->createGraphCloneEdgeless();
 
-        $superSource = $graphFlow->createVertex()->setLayoutAttribute('label', 's*');
-        $superSink   = $graphFlow->createVertex()->setLayoutAttribute('label', 't*');
+        $superSource = $graphFlow->createVertex();
+        $superSink   = $graphFlow->createVertex();
 
         $groups = $alg->getGroups();
         $groupA = $groups[0];

lib/Fhaculty/Graph/Algorithm/MinimumCostFlow/CycleCanceling.php

@@ -22,8 +22,8 @@ public function createGraph()
         // create resulting graph with supersource and supersink
         $resultGraph = $this->graph->createGraphClone();
 
-        $superSource = $resultGraph->createVertex()->setLayoutAttribute('label', 's*');
-        $superSink   = $resultGraph->createVertex()->setLayoutAttribute('label', 't*');
+        $superSource = $resultGraph->createVertex();
+        $superSink   = $resultGraph->createVertex();
 
         $sumBalance = 0;
 

lib/Fhaculty/Graph/Edge/Base.php

@@ -2,7 +2,6 @@
 
 namespace Fhaculty\Graph\Edge;
 
-use Fhaculty\Graph\Layoutable;
 use Fhaculty\Graph\Vertex;
 use Fhaculty\Graph\Set\Edges;
 use Fhaculty\Graph\Set\Vertices;
@@ -13,8 +12,10 @@
 use Fhaculty\Graph\Exception\UnderflowException;
 use Fhaculty\Graph\Exception\InvalidArgumentException;
 use Fhaculty\Graph\Exception\BadMethodCallException;
+use Fhaculty\Graph\Renderer\Layout;
+use Fhaculty\Graph\Renderer\LayoutAggregate;
 
-abstract class Base extends Layoutable implements VerticesAggregate
+abstract class Base implements VerticesAggregate, LayoutAggregate
 {
     /**
      * weight of this edge
@@ -40,6 +41,8 @@ abstract class Base extends Layoutable implements VerticesAggregate
      */
     protected $flow = NULL;
 
+    protected $layout = null;
+
     /**
      * get Vertices that are a target of this edge
      *
@@ -289,4 +292,13 @@ private function __clone()
         throw new BadMethodCallException();
         // @codeCoverageIgnoreEnd
     }
+
+    public function getLayout()
+    {
+        if ($this->layout === null) {
+            $this->layout = new Layout();
+        }
+
+        return $this->layout;
+    }
 }

lib/Fhaculty/Graph/Exporter/Dot.php

@@ -2,8 +2,8 @@
 
 namespace Fhaculty\Graph\Exporter;
 
-use Fhaculty\Graph\GraphViz;
 use Fhaculty\Graph\Graph;
+use Fhaculty\Graph\Renderer\GraphViz;
 
 class Dot implements ExporterInterface
 {

lib/Fhaculty/Graph/Exporter/Image.php

@@ -2,8 +2,8 @@
 
 namespace Fhaculty\Graph\Exporter;
 
-use Fhaculty\Graph\GraphViz;
 use Fhaculty\Graph\Graph;
+use Fhaculty\Graph\Renderer\GraphViz;
 
 class Image implements ExporterInterface
 {

lib/Fhaculty/Graph/Graph.php

@@ -20,9 +20,11 @@
 use Fhaculty\Graph\Set\Vertices;
 use Fhaculty\Graph\Set\VerticesMap;
 use Fhaculty\Graph\Set\Edges;
+use Fhaculty\Graph\Renderer\LayoutAggregate;
+use Fhaculty\Graph\Renderer\Layout;
 use Fhaculty\Graph\Set\DualAggregate;
 
-class Graph implements DualAggregate
+class Graph implements DualAggregate, LayoutAggregate
 {
     /**
      * @var ExporterInterface|null
@@ -36,6 +38,10 @@ class Graph implements DualAggregate
     protected $edgesStorage = array();
     protected $edges;
 
+    protected $layout = null;
+    protected $layoutVertex = null;
+    protected $layoutEdge = null;
+
     public function __construct()
     {
         $this->vertices = VerticesMap::factoryArrayReference($this->verticesStorage);
@@ -107,7 +113,7 @@ public function createVertexClone(Vertex $originalVertex)
         }
         $newVertex = new Vertex($id, $this);
         // TODO: properly set attributes of vertex
-        $newVertex->setLayout($originalVertex->getLayout());
+        $newVertex->getLayout()->setLayout($originalVertex->getLayout());
         $newVertex->setBalance($originalVertex->getBalance());
         $newVertex->setGroup($originalVertex->getGroup());
         $this->verticesStorage[$id] = $newVertex;
@@ -125,7 +131,7 @@ public function createVertexClone(Vertex $originalVertex)
     public function createGraphCloneEdgeless()
     {
         $graph = new Graph();
-//         $graph->setLayout($this->getLayout());
+        $graph->getLayout()->setLayout($this->getLayout());
         // TODO: set additional graph attributes
         foreach ($this->getVertices() as $originalVertex) {
             $vertex = $graph->createVertexClone($originalVertex);
@@ -244,7 +250,7 @@ private function createEdgeCloneInternal(Edge $originalEdge, $ia, $ib)
             $newEdge = $a->createEdge($b);
         }
         // TODO: copy edge attributes
-        $newEdge->setLayout($originalEdge->getLayout());
+        $newEdge->getLayout()->setLayout($originalEdge->getLayout());
         $newEdge->setWeight($originalEdge->getWeight());
         $newEdge->setFlow($originalEdge->getFlow());
         $newEdge->setCapacity($originalEdge->getCapacity());
@@ -489,7 +495,40 @@ public function __toString()
         return $this->getExporter()->getOutput($this);
     }
 
-    public function getLayout(){
-        return array();
+    public function getLayout()
+    {
+        if ($this->layout === null) {
+            $this->layout = new Layout();
+        }
+
+        return $this->layout;
+    }
+
+    /**
+     * get the global default layout for all vertices
+     *
+     * @return Layout
+     */
+    public function getLayoutVertexDefault()
+    {
+        if ($this->layoutVertex === null) {
+            $this->layoutVertex = new Layout();
+        }
+
+        return $this->layoutVertex;
+    }
+
+    /**
+     * get the global default layout for all edges
+     *
+     * @return Layout
+     */
+    public function getLayoutEdgeDefault()
+    {
+        if ($this->layoutEdge === null) {
+            $this->layoutEdge = new Layout();
+        }
+
+        return $this->layoutEdge;
     }
 }

lib/Fhaculty/Graph/GraphViz.php → lib/Fhaculty/Graph/Renderer/GraphViz.php

@@ -1,14 +1,16 @@
 <?php
 
-namespace Fhaculty\Graph;
+namespace Fhaculty\Graph\Renderer;
 
+use Fhaculty\Graph\Graph;
 use Fhaculty\Graph\Algorithm\Directed;
 use Fhaculty\Graph\Algorithm\Groups;
 use Fhaculty\Graph\Algorithm\Degree;
 use Fhaculty\Graph\Exception\UnexpectedValueException;
 use Fhaculty\Graph\Exception\InvalidArgumentException;
-use Fhaculty\Graph\Edge\Base as Edge;
 use \stdClass;
+use Fhaculty\Graph\Vertex;
+use Fhaculty\Graph\Edge\Base as Edge;
 
 class GraphViz
 {
@@ -26,9 +28,6 @@ class GraphViz
      */
     private $format = 'png';
 
-    private $layoutVertex = array();
-    private $layoutEdge = array();
-
     /**
      * Either the name of full path to GraphViz layout.
      *
@@ -147,50 +146,6 @@ public function display()
         // echo "... done\n";
     }
 
-    const LAYOUT_GRAPH = 1;
-    const LAYOUT_EDGE = 2;
-    const LAYOUT_VERTEX = 3;
-
-    private function mergeLayout(&$old, $new)
-    {
-        if ($new === NULL) {
-            $old = array();
-        } else {
-            foreach ($new as $key => $value) {
-                if ($value === NULL) {
-                    unset($old[$key]);
-                } else {
-                    $old[$key] = $value;
-                }
-            }
-        }
-    }
-
-    public function setLayout($where, $layout, $value = NULL)
-    {
-        if (!is_array($where)) {
-            $where = array($where);
-        }
-        if (func_num_args() > 2) {
-            $layout = array($layout => $value);
-        }
-        foreach ($where as $where) {
-            if ($where === self::LAYOUT_GRAPH) {
-                $this->graph->setLayout($layout, $value);
-            } elseif ($where === self::LAYOUT_EDGE) {
-                $this->mergeLayout($this->layoutEdge, $layout);
-            } elseif ($where === self::LAYOUT_VERTEX) {
-                $this->mergeLayout($this->layoutVertex, $layout);
-            } else {
-                throw new InvalidArgumentException('Invalid layout identifier');
-            }
-        }
-
-        return $this;
-    }
-
-    // end
-
     /**
      * create image file data contents for this graph
      *
@@ -285,15 +240,19 @@ public function createScript()
         $script = ($directed ? 'di':'') . 'graph G {' . self::EOL;
 
         // add global attributes
-        $layout = $this->graph->getLayout();
+        $layout = $this->graph->getLayout()->getAttributes();
         if ($layout) {
             $script .= $this->formatIndent . 'graph ' . $this->escapeAttributes($layout) . self::EOL;
         }
-        if ($this->layoutVertex) {
-            $script .= $this->formatIndent . 'node ' . $this->escapeAttributes($this->layoutVertex) . self::EOL;
+
+        $layout = $this->graph->getLayoutVertexDefault()->getAttributes();
+        if ($layout) {
+            $script .= $this->formatIndent . 'node ' . $this->escapeAttributes($layout) . self::EOL;
         }
-        if ($this->layoutEdge) {
-            $script .= $this->formatIndent . 'edge ' . $this->escapeAttributes($this->layoutEdge) . self::EOL;
+
+        $layout = $this->graph->getLayoutEdgeDefault()->getAttributes();
+        if ($layout) {
+            $script .= $this->formatIndent . 'edge ' . $this->escapeAttributes($layout) . self::EOL;
         }
 
         $alg = new Groups($this->graph);
@@ -428,7 +387,7 @@ public static function raw($string)
 
     protected function getLayoutVertex(Vertex $vertex)
     {
-        $layout = $vertex->getLayout();
+        $layout = $vertex->getLayout()->getAttributes();
 
         $balance = $vertex->getBalance();
         if($balance !== NULL){
@@ -446,7 +405,7 @@ protected function getLayoutVertex(Vertex $vertex)
 
     protected function getLayoutEdge(Edge $edge)
     {
-        $layout = $edge->getLayout();
+        $layout = $edge->getLayout()->getAttributes();
 
         // use flow/capacity/weight as edge label
         $label = NULL;

lib/Fhaculty/Graph/Layoutable.php → lib/Fhaculty/Graph/Renderer/Layout.php

@@ -1,10 +1,11 @@
 <?php
 
-namespace Fhaculty\Graph;
+namespace Fhaculty\Graph\Renderer;
 
 use Fhaculty\Graph\Exception\OutOfBoundsException;
+use Fhaculty\Graph\LayoutableInterface;
 
-abstract class Layoutable
+class Layout
 {
     /**
      * associative array of layout settings
@@ -18,7 +19,7 @@ abstract class Layoutable
      *
      * @return array
      */
-    public function getLayout()
+    public function getAttributes()
     {
         return $this->layout;
     }
@@ -28,9 +29,9 @@ public function getLayout()
      *
      * @param  array $attributes
      * @return self  $this (chainable)
-     * @see Layoutable::setLayoutAttribute()
+     * @see Layoutable::setAttribute()
      */
-    public function setLayout(array $attributes)
+    public function setAttributes(array $attributes)
     {
         foreach ($attributes as $key => $value) {
             if ($value === NULL) {
@@ -44,14 +45,14 @@ public function setLayout(array $attributes)
     }
 
     /**
-     * set a single layouto attribute
+     * set a single layout attribute
      *
      * @param  string $name
      * @param  string $value
      * @return self
-     * @see Layoutable::setLayout()
+     * @see Layoutable::setAttributes()
      */
-    public function setLayoutAttribute($name, $value)
+    public function setAttribute($name, $value)
     {
         if ($value === NULL) {
             unset($this->layout[$name]);
@@ -68,17 +69,24 @@ public function setLayoutAttribute($name, $value)
      * @param  string  $name
      * @return boolean
      */
-    public function hasLayoutAttribute($name)
+    public function hasAttribute($name)
     {
         return isset($this->layout[$name]);
     }
 
-    public function getLayoutAttribute($name)
+    public function getAttribute($name)
     {
         if (!isset($this->layout[$name])) {
             throw new OutOfBoundsException('Given layout attribute is not set');
         }
 
         return $this->layout[$name];
     }
+
+    public function setLayout(self $layout)
+    {
+        $this->layout = $layout->getAttributes();
+
+        return $this;
+    }
 }

lib/Fhaculty/Graph/Renderer/LayoutAggregate.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Fhaculty\Graph\Renderer;
+
+use Fhaculty\Graph\Exception\OutOfBoundsException;
+
+interface LayoutAggregate
+{
+    /**
+     *
+     * @return Layout
+     */
+    public function getLayout();
+}