Added products to main navigation

app/Http/Controllers/AccountController.php

@@ -475,16 +475,9 @@ class AccountController extends BaseController
      */
     private function showProducts()
     {
-        $columns = ['product', 'description', 'unit_cost'];
-        if (Auth::user()->account->invoice_item_taxes) {
-            $columns[] = 'tax_rate';
-        }
-        $columns[] = 'action';
-
         $data = [
             'account' => Auth::user()->account,
             'title' => trans('texts.product_library'),
-            'columns' => Utils::trans($columns),
         ];
 
         return View::make('accounts.products', $data);

app/Http/Controllers/ProductController.php

@@ -3,6 +3,7 @@
 use Auth;
 use URL;
 use View;
+use Utils;
 use Input;
 use Session;
 use Redirect;
@@ -22,7 +23,7 @@ class ProductController extends BaseController
 
     /**
      * ProductController constructor.
-     * 
+     *
      * @param ProductService $productService
      */
     public function __construct(ProductService $productService)
@@ -37,9 +38,34 @@ class ProductController extends BaseController
      */
     public function index()
     {
-        return Redirect::to('settings/' . ACCOUNT_PRODUCTS);
+        $columns = [
+            'checkbox',
+            'product',
+            'description',
+            'unit_cost'
+        ];
+
+        if (Auth::user()->account->invoice_item_taxes) {
+            $columns[] = 'tax_rate';
+        }
+        $columns[] = 'action';
+
+        return View::make('list', [
+            'entityType' => ENTITY_PRODUCT,
+            'title' => trans('texts.products'),
+            'sortCol' => '4',
+            'columns' => Utils::trans($columns),
+        ]);
     }
 
+    public function show($publicId)
+    {
+        Session::reflash();
+
+        return Redirect::to("products/$publicId/edit");
+    }
+
+
     /**
      * @return \Illuminate\Http\JsonResponse
      */
@@ -126,7 +152,7 @@ class ProductController extends BaseController
         $message = $productPublicId ? trans('texts.updated_product') : trans('texts.created_product');
         Session::flash('message', $message);
 
-        return Redirect::to('settings/' . ACCOUNT_PRODUCTS);
+        return Redirect::to(ACCOUNT_PRODUCTS);
     }
 
     /**
@@ -134,12 +160,12 @@ class ProductController extends BaseController
      */
     public function bulk()
     {
-        $action = Input::get('bulk_action');
+        $action = Input::get('action');
-        $ids = Input::get('bulk_public_id');
+        $ids = Input::get('public_id') ? Input::get('public_id') : Input::get('ids');
         $count = $this->productService->bulk($ids, $action);
 
         Session::flash('message', trans('texts.archived_product'));
 
-        return Redirect::to('settings/' . ACCOUNT_PRODUCTS);
+        return $this->returnBulk(ENTITY_PRODUCT, $action, $ids);
     }
 }

app/Http/routes.php

@@ -188,6 +188,11 @@ Route::group(['middleware' => 'auth:user'], function() {
     Route::get('api/credits/{client_id?}', ['as'=>'api.credits', 'uses'=>'CreditController@getDatatable']);
     Route::post('credits/bulk', 'CreditController@bulk');
 
+    Route::get('api/products', ['as'=>'api.products', 'uses'=>'ProductController@getDatatable']);
+    Route::resource('products', 'ProductController');
+    Route::post('products/bulk', 'ProductController@bulk');
+
+
     Route::get('/resend_confirmation', 'AccountController@resendConfirmation');
     Route::post('/update_setup', 'AppController@updateSetup');
 
@@ -230,10 +235,6 @@ Route::group([
     Route::resource('tokens', 'TokenController');
     Route::post('tokens/bulk', 'TokenController@bulk');
 
-    Route::get('api/products', ['as'=>'api.products', 'uses'=>'ProductController@getDatatable']);
-    Route::resource('products', 'ProductController');
-    Route::post('products/bulk', 'ProductController@bulk');
-
     Route::get('api/tax_rates', ['as'=>'api.tax_rates', 'uses'=>'TaxRateController@getDatatable']);
     Route::resource('tax_rates', 'TaxRateController');
     Route::post('tax_rates/bulk', 'TaxRateController@bulk');

app/Models/EntityModel.php

@@ -254,6 +254,7 @@ class EntityModel extends Eloquent
         $icons = [
             'dashboard' => 'tachometer',
             'clients' => 'users',
+            'products' => 'cube',
             'invoices' => 'file-pdf-o',
             'payments' => 'credit-card',
             'recurring_invoices' => 'files-o',

app/Ninja/Repositories/ProductRepository.php

@@ -19,13 +19,12 @@ class ProductRepository extends BaseRepository
 
     public function find($accountId)
     {
-        return DB::table('products')
+        $query = DB::table('products')
                 ->leftJoin('tax_rates', function($join) {
                     $join->on('tax_rates.id', '=', 'products.default_tax_rate_id')
                          ->whereNull('tax_rates.deleted_at');
                 })
                 ->where('products.account_id', '=', $accountId)
-                ->where('products.deleted_at', '=', null)
                 ->select(
                     'products.public_id',
                     'products.product_key',
@@ -35,6 +34,12 @@ class ProductRepository extends BaseRepository
                     'tax_rates.rate as tax_rate',
                     'products.deleted_at'
                 );
+
+        if (!\Session::get('show_trash:product')) {
+            $query->where('products.deleted_at', '=', null);
+        }
+
+        return $query;
     }
 
     public function save($data, $product = null)

app/Policies/ProductPolicy.php

@@ -10,20 +10,4 @@ use App\Models\User;
 class ProductPolicy extends EntityPolicy
 {
 
-    /**
+}
-     * @param User $user
-     * @param $item
-     * @return mixed
-     */
-	public static function edit(User $user, $item) {
-        return $user->hasPermission('admin');
-    }
-
-    /**
-     * @param User $user
-     * @return mixed
-     */
-    public static function create(User $user) {
-        return $user->hasPermission('admin');
-    }
-}

app/Services/ProductService.php

@@ -1,5 +1,7 @@
 <?php namespace App\Services;
 
+use Auth;
+use Utils;
 use App\Ninja\Repositories\ProductRepository;
 use App\Ninja\Datatables\ProductDatatable;
 
@@ -41,9 +43,13 @@ class ProductService extends BaseService
      */
     public function getDatatable($accountId)
     {
-        $datatable = new ProductDatatable(false);
+        $datatable = new ProductDatatable(true);
         $query = $this->productRepo->find($accountId);
 
+        if(!Utils::hasPermission('view_all')){
+            $query->where('products.user_id', '=', Auth::user()->id);
+        }
+
         return $this->datatableService->createDatatable($datatable, $query);
     }
 }

resources/lang/en/texts.php

@@ -287,7 +287,7 @@ $LANG = array(
     'view_as_recipient' => 'View as recipient',
     'product_library' => 'Product Library',
     'product' => 'Product',
-    'products' => 'Product Library',
+    'products' => 'Products',
     'fill_products' => 'Auto-fill products',
     'fill_products_help' => 'Selecting a product will automatically <b>fill in the description and cost</b>',
     'update_products' => 'Auto-update products',
@@ -2136,7 +2136,7 @@ $LANG = array(
     'update' => 'Update',
     'invoice_fields_help' => 'Drag and drop fields to change their order and location',
     'new_category' => 'New Category',
-
+    'restore_product' => 'Restore Product',
 
 );
 

resources/views/accounts/product.blade.php

@@ -1,13 +1,11 @@
 @extends('header')
 
-@section('content') 
+@section('content')
   @parent
 
-  @include('accounts.nav', ['selected' => ACCOUNT_PRODUCTS])
-
   {!! Former::open($url)->method($method)
       ->rules(['product_key' => 'required|max:255'])
-      ->addClass('warn-on-exit') !!}
+      ->addClass('col-md-10 col-md-offset-1 warn-on-exit') !!}
 
 
   <div class="panel panel-default">
@@ -22,7 +20,7 @@
   @endif
 
   {!! Former::text('product_key')->label('texts.product') !!}
-  {!! Former::textarea('notes') !!}
+  {!! Former::textarea('notes')->rows(6) !!}
   {!! Former::text('cost') !!}
 
   @if ($account->invoice_item_taxes)
@@ -35,8 +33,8 @@
   </div>
   </div>
 
-  {!! Former::actions( 
+  {!! Former::actions(
-      Button::normal(trans('texts.cancel'))->large()->asLinkTo(URL::to('/settings/products'))->appendIcon(Icon::create('remove-circle')),
+      Button::normal(trans('texts.cancel'))->large()->asLinkTo(URL::to('/products'))->appendIcon(Icon::create('remove-circle')),
       Button::success(trans('texts.save'))->submit()->large()->appendIcon(Icon::create('floppy-disk'))
   ) !!}
 
@@ -50,4 +48,4 @@
 
   </script>
 
-@stop
+@stop

resources/views/accounts/products.blade.php

@@ -1,6 +1,6 @@
 @extends('header')
 
-@section('content') 
+@section('content')
   @parent
 
   @include('accounts.nav', ['selected' => ACCOUNT_PRODUCTS])
@@ -13,7 +13,7 @@
   <div class="panel panel-default">
   <div class="panel-heading">
     <h3 class="panel-title">{!! trans('texts.product_settings') !!}</h3>
-  </div>  
+  </div>
   <div class="panel-body">
 
       {!! Former::checkbox('fill_products')->text(trans('texts.fill_products_help')) !!}
@@ -24,26 +24,4 @@
   </div>
   </div>
 
-  {!! Button::primary(trans('texts.create_product'))
+@stop
-        ->asLinkTo(URL::to('/products/create'))
-        ->withAttributes(['class' => 'pull-right'])
-        ->appendIcon(Icon::create('plus-sign')) !!}
-
-  @include('partials.bulk_form', ['entityType' => ENTITY_PRODUCT])
-
-  {!! Datatable::table()   
-      ->addColumn($columns)
-      ->setUrl(url('api/products/'))      
-      ->setOptions('sPaginationType', 'bootstrap')
-      ->setOptions('bFilter', false)      
-      ->setOptions('bAutoWidth', false)      
-      //->setOptions('aoColumns', [[ "sWidth"=> "15%" ], [ "sWidth"=> "35%" ]])
-      ->setOptions('aoColumnDefs', [['bSortable'=>false, 'aTargets'=>[3]]])
-      ->render('datatable') !!}
-
-  <script>
-    window.onDatatableReady = actionListHandler;
-  </script>  
-
-
-@stop

resources/views/header.blade.php

@@ -484,14 +484,15 @@
         @foreach ([
             'dashboard' => false,
             'clients' => false,
+            'products' => false,
+            'invoices' => false,
+            'payments' => false,
+            'recurring_invoices' => 'recurring',
             'credits' => false,
+            'quotes' => false,
             'tasks' => false,
             'expenses' => false,
             'vendors' => false,
-            'quotes' => false,
-            'invoices' => false,
-            'recurring_invoices' => 'recurring',
-            'payments' => false,
             'settings' => false,
         ] as $key => $value)
             {!! Form::nav_link($key, $value ?: $key) !!}
@@ -509,6 +510,7 @@
             @foreach([
                 'dashboard',
                 'clients',
+                'products',
                 'invoices',
                 'payments',
                 'recurring_invoices',