From aaa2fffa08da17f9316ddcb571fa10bfb5708e39 Mon Sep 17 00:00:00 2001
From: Kevin Papst <kpapst@gmx.net>
Date: Sun, 13 Nov 2016 22:19:18 +0100
Subject: [PATCH] form improvement using theme clone

---
 app/Resources/translations/messages.de.xliff  |  4 +
 app/Resources/views/admin/user.html.twig      |  4 +-
 .../views/default/_flash_messages.html.twig   |  9 --
 .../views/macros/datatables.html.twig         | 17 +++-
 app/config/config.yml                         |  6 +-
 app/config/parameters.yml.dist                |  7 +-
 composer.json                                 |  8 +-
 composer.lock                                 | 30 ++++---
 src/AppBundle/Form/Type/YesNoType.php         |  2 +
 src/AppBundle/Twig/Extensions.php             |  9 +-
 .../Controller/Admin/ProjectController.php    | 75 +++++++++++++++-
 .../DataFixtures/ORM/LoadFixtures.php         | 67 ++++++++++++--
 src/TimesheetBundle/Entity/Project.php        | 39 +++++++-
 src/TimesheetBundle/Form/ActivityEditForm.php | 11 ++-
 src/TimesheetBundle/Form/ProjectEditForm.php  | 88 +++++++++++++++++++
 .../Form/Type/CustomerType.php                | 44 ++++++++++
 .../Repository/ActivityRepository.php         |  2 +-
 .../Repository/ProjectRepository.php          | 11 ++-
 .../Repository/TimesheetRepository.php        |  8 +-
 .../Resources/views/admin/activity.html.twig  |  4 +-
 .../Resources/views/admin/project.html.twig   | 11 ++-
 .../views/admin/project_edit.html.twig        | 15 ++++
 22 files changed, 411 insertions(+), 60 deletions(-)
 create mode 100644 src/TimesheetBundle/Form/ProjectEditForm.php
 create mode 100644 src/TimesheetBundle/Form/Type/CustomerType.php
 create mode 100644 src/TimesheetBundle/Resources/views/admin/project_edit.html.twig

diff --git a/app/Resources/translations/messages.de.xliff b/app/Resources/translations/messages.de.xliff
index 9f948e00..ac6448d4 100644
--- a/app/Resources/translations/messages.de.xliff
+++ b/app/Resources/translations/messages.de.xliff
@@ -209,6 +209,10 @@
         <source>label.language</source>
         <target>Sprache</target>
       </trans-unit>
+      <trans-unit id="label.customer">
+        <source>label.customer</source>
+        <target>Kunde</target>
+      </trans-unit>
 
       <!--
         Buttons & Actions
diff --git a/app/Resources/views/admin/user.html.twig b/app/Resources/views/admin/user.html.twig
index 03086bdc..3ee31149 100644
--- a/app/Resources/views/admin/user.html.twig
+++ b/app/Resources/views/admin/user.html.twig
@@ -10,8 +10,8 @@
     {% if entries.count > 0 %}
         {{ tables.data_table_header({
             'label.id': '',
+            'label.alias': 'bookmark-o',
             'label.username': 'user',
-            'label.alias': 'comment-o',
             'label.email': 'envelope-o',
             'label.title': 'graduation-cap',
             'label.active': 'lock',
@@ -22,8 +22,8 @@
         {% for entry in entries %}
             <tr>
                 <td>{{ entry.id }}</td>
-                <td><a href="{{ path('user_profile', {'username' : entry.username}) }}">{{ entry.username }}</a></td>
                 <td>{{ entry.alias }}</td>
+                <td>{{ entry.username }}</td>
                 <td>{{ entry.email }}</td>
                 <td>{{ entry.title }}</td>
                 <td>{{ widgets.label_visible(entry.active) }}</td>
diff --git a/app/Resources/views/default/_flash_messages.html.twig b/app/Resources/views/default/_flash_messages.html.twig
index f2845a2f..2ea971e1 100644
--- a/app/Resources/views/default/_flash_messages.html.twig
+++ b/app/Resources/views/default/_flash_messages.html.twig
@@ -1,12 +1,3 @@
-{#
-   This is a template fragment designed to be included in other templates
-   See http://symfony.com/doc/current/book/templating.html#including-other-templates
-
-   A common practice to better distinguish between templates and fragments is to
-   prefix fragments with an underscore. That's why this template is called
-   '_flash_messages.html.twig' instead of 'flash_messages.html.twig'
-#}
-
 {% if app.session.started and app.session.flashBag.peekAll is not empty %}
     <div class="messages">
         {% for type, messages in app.session.flashBag.all %}
diff --git a/app/Resources/views/macros/datatables.html.twig b/app/Resources/views/macros/datatables.html.twig
index eb0e6989..32b47b56 100644
--- a/app/Resources/views/macros/datatables.html.twig
+++ b/app/Resources/views/macros/datatables.html.twig
@@ -1,9 +1,20 @@
 
 {% macro data_table_header(entries) %}
 <div class="box">
-    {# <div class="box-header">
-        <h3 class="box-title">Hover Data Table</h3>
-    </div> #}
+    {#
+    <div class="box-header">
+        <h3 class="box-title"><a href="#" class="btn btn-primary">jkbkj</a></h3>
+        <div class="box-tools">
+            <ul class="pagination pagination-sm no-margin pull-right">
+                <li><a href="#">«</a></li>
+                <li><a href="#">1</a></li>
+                <li><a href="#">2</a></li>
+                <li><a href="#">3</a></li>
+                <li><a href="#">»</a></li>
+            </ul>
+        </div>
+    </div>
+    #}
     <div class="box-body no-padding">
         <div class="dataTables_wrapper form-inline dt-bootstrap">
             <div class="row">
diff --git a/app/config/config.yml b/app/config/config.yml
index a7a4f4bd..22a23d2a 100644
--- a/app/config/config.yml
+++ b/app/config/config.yml
@@ -54,9 +54,9 @@ framework:
 twig:
     debug:            "%kernel.debug%"
     strict_variables: "%kernel.debug%"
-    form_themes:
-        - "bootstrap_3_layout.html.twig"
-        - "form/fields.html.twig"
+#    form_themes:
+#        - "bootstrap_3_layout.html.twig"
+#        - "form/fields.html.twig"
     globals:
         kimai_context:
             date_1: "d.m.Y"     # used for display in timesheets
diff --git a/app/config/parameters.yml.dist b/app/config/parameters.yml.dist
index 4a499479..3230c1c7 100644
--- a/app/config/parameters.yml.dist
+++ b/app/config/parameters.yml.dist
@@ -1,6 +1,5 @@
 # This file defines the canonical configuration parameters of the application.
-# Symfony uses this file as a template to generate the real app/config/parameters.yml
-# used by the application.
+# Symfony uses this file as a template to generate the real app/config/parameters.yml used by the application.
 # See http://symfony.com/doc/current/best_practices/configuration.html#canonical-parameters
 parameters:
 
@@ -28,8 +27,8 @@ parameters:
     mailer_password:   ~
     mailer_from:       anonymous@example.com
 
-    # The code of the default language used by the application ('en' = English)
-    locale: en
+    # The code of the default language used by the application ('en' = English, 'de' = German)
+    locale: de
 
     # The 'secret' value is a random string of characters, numbers and symbols
     # used internally by Symfony in several places (CSRF tokens, URI signing,
diff --git a/composer.json b/composer.json
index 2cc71d38..9f1bfbf6 100644
--- a/composer.json
+++ b/composer.json
@@ -12,6 +12,12 @@
             "app/AppCache.php"
         ]
     },
+    "repositories": [
+        {
+            "type": "vcs",
+            "url": "https://github.com/kevinpapst/AdminThemeBundle"
+        }
+    ],
     "require": {
         "php": ">=5.5.9",
         "ext-pdo_sqlite": "*",
@@ -31,7 +37,7 @@
         "symfony/symfony": "^3.1",
         "twig/extensions": "^1.3",
         "white-october/pagerfanta-bundle": "^1.0",
-        "avanzu/admin-theme-bundle": "dev-feature/2.0",
+        "avanzu/admin-theme-bundle": "dev-feature/2.0-advanced",
         "beberlei/DoctrineExtensions": "^1.0"
     },
     "require-dev": {
diff --git a/composer.lock b/composer.lock
index e8a9e7d8..66b9449a 100644
--- a/composer.lock
+++ b/composer.lock
@@ -4,21 +4,21 @@
         "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file",
         "This file is @generated automatically"
     ],
-    "hash": "d5aa01779571420f9166199f60fa2a17",
-    "content-hash": "20dda7356ab7c685ada168dd7e9b7593",
+    "hash": "6823aa58cef4fcd853f97ddd207c7cbd",
+    "content-hash": "a88a2b7ecf31f6e57be9ee123efac828",
     "packages": [
         {
             "name": "almasaeed2010/adminlte",
-            "version": "v2.3.7",
+            "version": "v2.3.8",
             "source": {
                 "type": "git",
                 "url": "https://github.com/almasaeed2010/AdminLTE.git",
-                "reference": "797b65b3ca90525a4e048de5b8d02ba6878ea239"
+                "reference": "ced14b133daed8c163863a497c43dcf69eea4340"
             },
             "dist": {
                 "type": "zip",
-                "url": "https://api.github.com/repos/almasaeed2010/AdminLTE/zipball/797b65b3ca90525a4e048de5b8d02ba6878ea239",
-                "reference": "797b65b3ca90525a4e048de5b8d02ba6878ea239",
+                "url": "https://api.github.com/repos/almasaeed2010/AdminLTE/zipball/ced14b133daed8c163863a497c43dcf69eea4340",
+                "reference": "ced14b133daed8c163863a497c43dcf69eea4340",
                 "shasum": ""
             },
             "type": "library",
@@ -45,20 +45,20 @@
                 "theme",
                 "web"
             ],
-            "time": "2016-10-19 15:34:25"
+            "time": "2016-11-04 14:42:55"
         },
         {
             "name": "avanzu/admin-theme-bundle",
-            "version": "dev-feature/2.0",
+            "version": "dev-feature/2.0-advanced",
             "source": {
                 "type": "git",
-                "url": "https://github.com/avanzu/AdminThemeBundle.git",
-                "reference": "b2728cd76434cafd5f37dd0b75fe133429d2b1e0"
+                "url": "https://github.com/kevinpapst/AdminThemeBundle.git",
+                "reference": "9332a142b271d5eccd0d69e71de6b290cc133ae8"
             },
             "dist": {
                 "type": "zip",
-                "url": "https://api.github.com/repos/avanzu/AdminThemeBundle/zipball/b2728cd76434cafd5f37dd0b75fe133429d2b1e0",
-                "reference": "b2728cd76434cafd5f37dd0b75fe133429d2b1e0",
+                "url": "https://api.github.com/repos/kevinpapst/AdminThemeBundle/zipball/9332a142b271d5eccd0d69e71de6b290cc133ae8",
+                "reference": "9332a142b271d5eccd0d69e71de6b290cc133ae8",
                 "shasum": ""
             },
             "require": {
@@ -80,7 +80,6 @@
                     "Avanzu\\AdminThemeBundle\\": ""
                 }
             },
-            "notification-url": "https://packagist.org/downloads/",
             "license": [
                 "MIT"
             ],
@@ -91,7 +90,10 @@
                 }
             ],
             "description": "Admin Theme based on the AdminLTE Template for easy integration into symfony",
-            "time": "2016-08-26 08:28:19"
+            "support": {
+                "source": "https://github.com/kevinpapst/AdminThemeBundle/tree/feature/2.0-advanced"
+            },
+            "time": "2016-11-13 10:50:00"
         },
         {
             "name": "beberlei/DoctrineExtensions",
diff --git a/src/AppBundle/Form/Type/YesNoType.php b/src/AppBundle/Form/Type/YesNoType.php
index da0b331d..d649ec50 100644
--- a/src/AppBundle/Form/Type/YesNoType.php
+++ b/src/AppBundle/Form/Type/YesNoType.php
@@ -30,6 +30,8 @@ class YesNoType extends AbstractType
     {
         $resolver->setDefaults([
             'choices' => ['yes' => true, 'no' => false],
+            'multiple' => false,
+            'expanded' => true,
         ]);
     }
 
diff --git a/src/AppBundle/Twig/Extensions.php b/src/AppBundle/Twig/Extensions.php
index 47714b29..061a9fd5 100644
--- a/src/AppBundle/Twig/Extensions.php
+++ b/src/AppBundle/Twig/Extensions.php
@@ -82,22 +82,23 @@ class Extensions extends \Twig_Extension
         $minute = $minute > 9 ? $minute : '0' . $minute;
 
         if (!$includeSeconds) {
-            return $hour . ':' . $minute;
+            return $hour . ':' . $minute . ' h';
         }
 
         $second = $seconds % 60;
         $second = $second > 9 ? $second : '0' . $second;
 
-        return $hour . ':' . $minute  . ':' . $second . 'h';
+        return $hour . ':' . $minute  . ':' . $second . ' h';
     }
 
     /**
      * @param float $amount
+     * @param string $currency
      * @return string
      */
-    public function money($amount)
+    public function money($amount, $currency = 'EUR')
     {
-        return round($amount) . ' €';
+        return round($amount) . ' ' . Intl::getCurrencyBundle()->getCurrencySymbol($currency);
     }
 
     /**
diff --git a/src/TimesheetBundle/Controller/Admin/ProjectController.php b/src/TimesheetBundle/Controller/Admin/ProjectController.php
index 7be428c2..1c8fec7f 100644
--- a/src/TimesheetBundle/Controller/Admin/ProjectController.php
+++ b/src/TimesheetBundle/Controller/Admin/ProjectController.php
@@ -12,13 +12,15 @@
 namespace TimesheetBundle\Controller\Admin;
 
 use Pagerfanta\Pagerfanta;
+use Symfony\Component\HttpFoundation\Request;
 use TimesheetBundle\Entity\Project;
-use TimesheetBundle\Entity\Timesheet;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Cache;
+use TimesheetBundle\Form\ProjectEditForm;
+use TimesheetBundle\Repository\ProjectRepository;
 
 /**
  * Controller used to manage projects in the admin part of the site.
@@ -35,6 +37,9 @@ class ProjectController extends Controller
      * @Route("/page/{page}", requirements={"page": "[1-9]\d*"}, name="admin_project_paginated")
      * @Method("GET")
      * @Cache(smaxage="10")
+     *
+     * @param $page
+     * @return \Symfony\Component\HttpFoundation\Response
      */
     public function indexAction($page)
     {
@@ -43,4 +48,72 @@ class ProjectController extends Controller
 
         return $this->render('TimesheetBundle:admin:project.html.twig', ['entries' => $entries]);
     }
+
+    /**
+     * @Route("/{id}/edit", name="admin_project_edit")
+     * @Method({"GET", "POST"})
+     *
+     * @param $id
+     * @param Request $request
+     * @return \Symfony\Component\HttpFoundation\RedirectResponse|\Symfony\Component\HttpFoundation\Response
+     */
+    public function editAction($id, Request $request)
+    {
+        $project = $this->getById($id);
+        $editForm = $this->createEditForm($project);
+
+        $editForm->handleRequest($request);
+
+        if ($editForm->isSubmitted() && $editForm->isValid()) {
+            $entityManager = $this->getDoctrine()->getManager();
+            $entityManager->persist($project);
+            $entityManager->flush();
+
+            $this->addFlash('success', 'action.updated_successfully');
+
+            return $this->redirectToRoute(
+                'admin_project', ['id' => $project->getId()]
+            );
+        }
+
+        return $this->render(
+            'TimesheetBundle:admin:project_edit.html.twig',
+            [
+                'project' => $project,
+                'form' => $editForm->createView()
+            ]
+        );
+    }
+
+    /**
+     * @param $id
+     * @return null|Project
+     */
+    protected function getById($id)
+    {
+        /* @var $repo ProjectRepository */
+        $repo = $this->getDoctrine()->getRepository(Project::class);
+        $activity = $repo->getById($id);
+        if (null === $activity) {
+            throw new NotFoundHttpException('Project "'.$id.'" does not exist');
+        }
+        return $activity;
+    }
+
+    /**
+     * @param Project $project
+     * @return \Symfony\Component\Form\Form
+     */
+    private function createEditForm(Project $project)
+    {
+        return $this->createForm(
+            ProjectEditForm::class,
+            $project,
+            [
+                'action' => $this->generateUrl('admin_project_edit', ['id' => $project->getId()]),
+                'method' => 'POST',
+                'currency' => $project->getCurrency()
+            ]
+        );
+    }
 }
diff --git a/src/TimesheetBundle/DataFixtures/ORM/LoadFixtures.php b/src/TimesheetBundle/DataFixtures/ORM/LoadFixtures.php
index 238ece33..45325689 100644
--- a/src/TimesheetBundle/DataFixtures/ORM/LoadFixtures.php
+++ b/src/TimesheetBundle/DataFixtures/ORM/LoadFixtures.php
@@ -12,6 +12,7 @@
 namespace TimesheetBundle\DataFixtures\ORM;
 
 use AppBundle\Entity\User;
+use Symfony\Component\Intl\Intl;
 use TimesheetBundle\Entity\Activity;
 use TimesheetBundle\Entity\Customer;
 use TimesheetBundle\Entity\Project;
@@ -173,6 +174,7 @@ class LoadFixtures extends AppBundleLoadFixtures
 
             $entry = new Project();
             $entry->setName($this->getRandomProject());
+            $entry->setCurrency($this->getRandomCurrency());
             $entry->setBudget(rand(1000, 100000));
             $entry->setComment($this->getRandomPhrase());
             $entry->setCustomer($allCustomer[rand(1, $amountCustomer)]);
@@ -202,6 +204,9 @@ class LoadFixtures extends AppBundleLoadFixtures
         $manager->flush();
     }
 
+    /**
+     * @return string[]
+     */
     private function getActivities()
     {
         return [
@@ -213,33 +218,51 @@ class LoadFixtures extends AppBundleLoadFixtures
             'Internal',
             'Research',
             'Meeting',
+            'Hosting',
+            'Relaunch',
+            'Support',
+            'Refactoring',
+            'Interview',
         ];
     }
 
+    /**
+     * @return string
+     */
     private function getRandomActivity()
     {
         $all = $this->getActivities();
         return $all[array_rand($all)];
     }
 
+    /**
+     * @return string[]
+     */
     private function getProjects()
     {
         return [
-            'FooBar',
-            'Relaunch',
-            'Refactoring',
+            'User Experience',
+            'Database Migration',
             'Test Automatisation',
-            'Website redesign',
-            'Services',
+            'Website Redesign',
+            'API Development',
+            'Hosting & Server',
+            'Customer Relations',
         ];
     }
 
+    /**
+     * @return string
+     */
     private function getRandomProject()
     {
         $all = $this->getProjects();
         return $all[array_rand($all)];
     }
 
+    /**
+     * @return string[]
+     */
     private function getLocations()
     {
         return [
@@ -258,12 +281,18 @@ class LoadFixtures extends AppBundleLoadFixtures
         ];
     }
 
+    /**
+     * @return string
+     */
     private function getRandomLocation()
     {
         $all = $this->getLocations();
         return $all[array_rand($all)];
     }
 
+    /**
+     * @return string[]
+     */
     private function getCustomers()
     {
         return [
@@ -280,9 +309,37 @@ class LoadFixtures extends AppBundleLoadFixtures
         ];
     }
 
+    /**
+     * @return string
+     */
     private function getRandomCustomer()
     {
         $all = $this->getCustomers();
         return $all[array_rand($all)];
     }
+
+    /**
+     * @return string[]
+     */
+    private function getCurrencies()
+    {
+        return [
+            'EUR',
+            'GBP',
+            'USD',
+            'RUB',
+            'JPY',
+            'CNY',
+            'INR'
+        ];
+    }
+
+    /**
+     * @return string
+     */
+    private function getRandomCurrency()
+    {
+        $all = $this->getCurrencies();
+        return $all[array_rand($all)];
+    }
 }
diff --git a/src/TimesheetBundle/Entity/Project.php b/src/TimesheetBundle/Entity/Project.php
index b250eb3c..2eca9ecc 100644
--- a/src/TimesheetBundle/Entity/Project.php
+++ b/src/TimesheetBundle/Entity/Project.php
@@ -68,10 +68,17 @@ class Project
      */
     private $budget = 0.00;
 
+    /**
+     * @var string
+     *
+     * @ORM\Column(name="currency", type="string", length=3, nullable=false)
+     */
+    private $currency = 'EUR';
+
     /**
      * @var Activity[]
      *
-     * @ORM\OneToMany(targetEntity="TimesheetBundle\Entity\Activity", mappedBy="project")
+     * @ORM\OneToMany(targetEntity="TimesheetBundle\Entity\Activity", mappedBy="project", cascade={"persist", "merge", "remove"})
      */
     private $activities;
 
@@ -85,6 +92,25 @@ class Project
         return $this->id;
     }
 
+    /**
+     * @return string
+     */
+    public function getCurrency()
+    {
+        return $this->currency;
+    }
+
+    /**
+     * @param string $currency
+     * @return $this
+     */
+    public function setCurrency($currency)
+    {
+        $this->currency = $currency;
+
+        return $this;
+    }
+
     /**
      * @return Customer
      */
@@ -199,6 +225,17 @@ class Project
         return $this->budget;
     }
 
+    /**
+     * @param Activity[] $activities
+     * @return $this
+     */
+    public function setActivities($activities)
+    {
+        $this->activities = $activities;
+
+        return $this;
+    }
+
     /**
      * @return Activity[]
      */
diff --git a/src/TimesheetBundle/Form/ActivityEditForm.php b/src/TimesheetBundle/Form/ActivityEditForm.php
index 6831eca7..14df5c7d 100644
--- a/src/TimesheetBundle/Form/ActivityEditForm.php
+++ b/src/TimesheetBundle/Form/ActivityEditForm.php
@@ -14,6 +14,7 @@ namespace TimesheetBundle\Form;
 use AppBundle\Form\Type\YesNoType;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\TextareaType;
+use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use TimesheetBundle\Entity\Activity;
@@ -34,20 +35,22 @@ class ActivityEditForm extends AbstractType
     {
         $builder
             // string - length 255
-            ->add('name', null, [
+            ->add('name', TextType::class, [
                 'label' => 'label.name',
             ])
             // text
             ->add('comment', TextareaType::class, [
                 'label' => 'label.comment',
+                'required' => false,
+            ])
+            // entity type: project
+            ->add('project', ProjectType::class, [
+                'label' => 'label.project',
             ])
             // boolean
             ->add('visible', YesNoType::class, [
                 'label' => 'label.visible',
             ])
-            ->add('project', ProjectType::class, [
-                'label' => 'label.project',
-            ])
         ;
     }
 
diff --git a/src/TimesheetBundle/Form/ProjectEditForm.php b/src/TimesheetBundle/Form/ProjectEditForm.php
new file mode 100644
index 00000000..2d83d1ed
--- /dev/null
+++ b/src/TimesheetBundle/Form/ProjectEditForm.php
@@ -0,0 +1,88 @@
+<?php
+
+/*
+ * This file is part of the Kimai package.
+ *
+ * (c) Kevin Papst <kevin@kevinpapst.de>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace TimesheetBundle\Form;
+
+use AppBundle\Form\Type\YesNoType;
+use Symfony\Bridge\Doctrine\Form\Type\EntityType;
+use Symfony\Component\Form\AbstractType;
+use Symfony\Component\Form\Extension\Core\Type\MoneyType;
+use Symfony\Component\Form\Extension\Core\Type\TextareaType;
+use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\Intl\Intl;
+use Symfony\Component\OptionsResolver\OptionsResolver;
+use TimesheetBundle\Entity\Project;
+use TimesheetBundle\Form\Type\CustomerType;
+
+/**
+ * Defines the form used to manipulate Projects.
+ *
+ * @author Kevin Papst <kevin@kevinpapst.de>
+ */
+class ProjectEditForm extends AbstractType
+{
+
+    /**
+     * {@inheritdoc}
+     */
+    public function buildForm(FormBuilderInterface $builder, array $options)
+    {
+        $builder
+            // string - length 255
+            ->add('name', null, [
+                'label' => 'label.name',
+            ])
+            // text
+            ->add('comment', TextareaType::class, [
+                'label' => 'label.comment',
+            ])
+            // customer
+            ->add('customer', CustomerType::class, [
+                'label' => 'label.customer',
+            ])
+            // boolean
+            ->add('visible', YesNoType::class, [
+                'label' => 'label.visible',
+            ])
+            // string
+            ->add('budget', MoneyType::class, [
+                'label' => 'label.budget',
+                'currency' => $builder->getOption('currency'),
+            ])
+            // FIXME add budget
+            // do not allow activity selection as this causes headaches:
+            // 1. it is a bad UX
+            // 2. what should happen if they are detached?
+            /*
+            ->add('activities', EntityType::class, [
+                'label' => 'label.activity',
+                'class' => 'TimesheetBundle:Activity',
+                'multiple' => true,
+                'expanded' => true
+            ])
+            */
+        ;
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function configureOptions(OptionsResolver $resolver)
+    {
+        $resolver->setDefaults([
+            'data_class' => Project::class,
+            'csrf_protection' => true,
+            'csrf_field_name' => '_token',
+            'csrf_token_id' => 'admin_activity_edit',
+            'currency' => 'EUR,'
+        ]);
+    }
+}
diff --git a/src/TimesheetBundle/Form/Type/CustomerType.php b/src/TimesheetBundle/Form/Type/CustomerType.php
new file mode 100644
index 00000000..1f70ff90
--- /dev/null
+++ b/src/TimesheetBundle/Form/Type/CustomerType.php
@@ -0,0 +1,44 @@
+<?php
+
+/*
+ * This file is part of the Kimai package.
+ *
+ * (c) Kevin Papst <kevin@kevinpapst.de>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace TimesheetBundle\Form\Type;
+
+use Symfony\Bridge\Doctrine\Form\Type\EntityType;
+use Symfony\Component\Form\AbstractType;
+use Symfony\Component\OptionsResolver\OptionsResolver;
+
+/**
+ * Custom form field type to select a customer.
+ *
+ * @author Kevin Papst <kevin@kevinpapst.de>
+ */
+class CustomerType extends AbstractType
+{
+
+    /**
+     * {@inheritdoc}
+     */
+    public function configureOptions(OptionsResolver $resolver)
+    {
+        $resolver->setDefaults([
+            'class' => 'TimesheetBundle:Customer',
+            'choice_label' => 'name',
+        ]);
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getParent()
+    {
+        return EntityType::class;
+    }
+}
diff --git a/src/TimesheetBundle/Repository/ActivityRepository.php b/src/TimesheetBundle/Repository/ActivityRepository.php
index 0bbc2d74..9423619a 100644
--- a/src/TimesheetBundle/Repository/ActivityRepository.php
+++ b/src/TimesheetBundle/Repository/ActivityRepository.php
@@ -100,7 +100,7 @@ class ActivityRepository extends EntityRepository
      */
     public function findAll($page = 1)
     {
-        return $this->getPager($this->queryLatest(), $page);
+        return $this->getPager($this->queryAll(), $page);
     }
 
     /**
diff --git a/src/TimesheetBundle/Repository/ProjectRepository.php b/src/TimesheetBundle/Repository/ProjectRepository.php
index dc801d59..da94fdf3 100644
--- a/src/TimesheetBundle/Repository/ProjectRepository.php
+++ b/src/TimesheetBundle/Repository/ProjectRepository.php
@@ -12,7 +12,7 @@
 namespace TimesheetBundle\Repository;
 
 use AppBundle\Entity\User;
-use TimesheetBundle\Entity\Timesheet;
+use TimesheetBundle\Entity\Project;
 use Doctrine\ORM\EntityRepository;
 use Doctrine\ORM\Query;
 use Pagerfanta\Adapter\DoctrineORMAdapter;
@@ -27,6 +27,15 @@ use TimesheetBundle\Model\ProjectStatistic;
 class ProjectRepository extends EntityRepository
 {
 
+    /**
+     * @param $id
+     * @return null|Project
+     */
+    public function getById($id)
+    {
+        return $this->find($id);
+    }
+
     /**
      * Return statistic data for all user.
      *
diff --git a/src/TimesheetBundle/Repository/TimesheetRepository.php b/src/TimesheetBundle/Repository/TimesheetRepository.php
index e8cdc81e..3d9bc8a7 100644
--- a/src/TimesheetBundle/Repository/TimesheetRepository.php
+++ b/src/TimesheetBundle/Repository/TimesheetRepository.php
@@ -39,8 +39,10 @@ class TimesheetRepository extends EntityRepository
      */
     protected function queryThisMonth($select, User $user = null)
     {
-        $end = new DateTime();
-        $begin = $end->sub(new \DateInterval("P30D"));  // FIXME last month is not last 30 days
+        $end = new DateTime('last day of this month');
+        $end->setTime(23, 59, 59);
+        $begin = new DateTime('first day of this month');
+        $begin->setTime(0,0,0);
 
         return $this->queryTimeRange($select, $begin, $end, $user);
     }
@@ -59,7 +61,7 @@ class TimesheetRepository extends EntityRepository
         $qb->select($select)
             ->from('TimesheetBundle:Timesheet', 't')
             ->where($qb->expr()->gt('t.begin', ':from'))
-            ->andWhere($qb->expr()->gt('t.end', ':to'))
+            ->andWhere($qb->expr()->lt('t.end', ':to'))
             ->setParameter('from', $begin, Type::DATETIME)
             ->setParameter('to', $end, Type::DATETIME);
 
diff --git a/src/TimesheetBundle/Resources/views/admin/activity.html.twig b/src/TimesheetBundle/Resources/views/admin/activity.html.twig
index f30eb325..e66b31e2 100644
--- a/src/TimesheetBundle/Resources/views/admin/activity.html.twig
+++ b/src/TimesheetBundle/Resources/views/admin/activity.html.twig
@@ -28,8 +28,8 @@
                 <td>
                     {{ widgets.button_group({
                         'edit': path('admin_activity_edit', {'id': entry.id}),
-                        'trash': '#'}
-                    ) }}
+                        'trash': '#'
+                    }) }}
                 </td>
             </tr>
         {% endfor %}
diff --git a/src/TimesheetBundle/Resources/views/admin/project.html.twig b/src/TimesheetBundle/Resources/views/admin/project.html.twig
index b18086f0..3eb883e2 100644
--- a/src/TimesheetBundle/Resources/views/admin/project.html.twig
+++ b/src/TimesheetBundle/Resources/views/admin/project.html.twig
@@ -6,10 +6,13 @@
 {% block page_subtitle %}{{ 'admin_project.subtitle'|trans }}{% endblock %}
 
 {% block main %}
+    {{ include('default/_flash_messages.html.twig') }}
+
     {% if entries.count > 0 %}
         {{ tables.data_table_header({
             'label.id': '',
             'label.name': 'bookmark-o',
+            'label.customer': 'address-card-o',
             'label.comment': 'comment-o',
             'label.activity': 'tasks',
             'label.budget': 'credit-card',
@@ -21,12 +24,16 @@
             <tr>
                 <td>{{ entry.id }}</td>
                 <td>{{ entry.name }}</td>
+                <td>{{ entry.customer.name }}</td>
                 <td>{{ entry.comment }}</td>
                 <td>{{ widgets.badge_counter(entry.activities.count) }}</td>
-                <td>{{ entry.budget|money}}</td>
+                <td>{{ entry.budget|money(entry.currency) }}</td>
                 <td>{{ widgets.label_visible(entry.visible) }}</td>
                 <td>
-                    {{ widgets.button_group({'edit': '#', 'trash': '#'}) }}
+                    {{ widgets.button_group({
+                        'edit': path('admin_project_edit', {'id': entry.id}),
+                        'trash': '#'
+                    }) }}
                 </td>
             </tr>
         {% endfor %}
diff --git a/src/TimesheetBundle/Resources/views/admin/project_edit.html.twig b/src/TimesheetBundle/Resources/views/admin/project_edit.html.twig
new file mode 100644
index 00000000..eaf6c7d3
--- /dev/null
+++ b/src/TimesheetBundle/Resources/views/admin/project_edit.html.twig
@@ -0,0 +1,15 @@
+{% extends 'base.html.twig' %}
+{% import "macros/widgets.html.twig" as widgets %}
+{% import "macros/datatables.html.twig" as tables %}
+
+{% block page_title %}{{ 'admin_project.title'|trans }}{% endblock %}
+{% block page_subtitle %}{{ 'admin_project.subtitle'|trans }}{% endblock %}
+
+{% block main %}
+    {{ include('default/_flash_messages.html.twig') }}
+    {{ include('default/_form.html.twig', {
+        'title': project.name,
+        'form': form,
+        'back': path('admin_project')
+    }) }}
+{% endblock %}