Merge remote-tracking branch 'origin/develop'

# Conflicts:
#	config/version.php

app/Console/Commands/LdapSync.php

@@ -191,21 +191,27 @@ class LdapSync extends Command
         ];
         // Only update the database if is not a dry run
         if (!$this->dryrun) {
-            if ($user->save()) {
-                $summary['note']   = ($user->wasRecentlyCreated ? 'CREATED' : 'UPDATED');
-                $summary['status'] = 'SUCCESS';
-            } else {
-                $errors = '';
-                foreach ($user->getErrors()->getMessages() as  $error) {
-                    $errors .= implode(", ",$error);
+            if ($user->isDirty()) { //if nothing on the user changed, don't bother trying to save anything nor put anything in the summary
+                if ($user->save()) {
+                    $summary['note']   = ($user->wasRecentlyCreated ? 'CREATED' : 'UPDATED');
+                    $summary['status'] = 'SUCCESS';
+                } else {
+                    $errors = '';
+                    foreach ($user->getErrors()->getMessages() as  $error) {
+                        $errors .= implode(", ",$error);
+                    }
+                    $summary['note']   = $snipeUser->getDN().' was not imported. REASON: '.$errors;
+                    $summary['status'] = 'ERROR';
                 }
-                $summary['note']   = $snipeUser->getDN().' was not imported. REASON: '.$errors;
-                $summary['status'] = 'ERROR';
+            } else {
+                $summary = null;
             }
         }
 
         // $summary['note'] = ($user->getOriginal('username') ? 'UPDATED' : 'CREATED'); // this seems, kinda, like, superfluous, relative to the $summary['note'] thing above, yeah?
-        $this->summary->push($summary);
+        if($summary) { //if the $user wasn't dirty, $summary was set to null so that we will skip the following push()
+            $this->summary->push($summary);
+        }
     }
 
     /**

app/Http/Controllers/Api/ReportsController.php

@@ -49,6 +49,7 @@ class ReportsController extends Controller
             'created_at',
             'target_id',
             'user_id',
+            'accept_signature',
             'action_type',
             'note'
         ];

app/Http/Controllers/ManufacturersController.php

@@ -70,7 +70,7 @@ class ManufacturersController extends Controller
         $manufacturer->support_url      = $request->input('support_url');
         $manufacturer->support_phone    = $request->input('support_phone');
         $manufacturer->support_email    = $request->input('support_email');
-        $manufacturer = $request->handleImages($manufacturer,600, public_path().'/uploads/manufacturers');
+        $manufacturer = $request->handleImages($manufacturer);
 
 
 
@@ -137,7 +137,7 @@ class ManufacturersController extends Controller
             $manufacturer->image = null;
         }
 
-        $manufacturer = $request->handleImages($manufacturer,600, public_path().'/uploads/manufacturers');
+        $manufacturer = $request->handleImages($manufacturer);
 
 
         if ($manufacturer->save()) {

app/Http/Controllers/ReportsController.php

@@ -682,7 +682,7 @@ class ReportsController extends Controller
                             $diff = ($asset->purchase_cost - $depreciation);
                             $row[]        = Helper::formatCurrencyOutput($depreciation);
                             $row[]        = Helper::formatCurrencyOutput($diff);
-                            $row[]        = ($asset->depreciated_date()!='') ? $asset->depreciated_date()->format('Y-m-d') : '';
+                            $row[]        = ($asset->depreciation) ? $asset->depreciated_date()->format('Y-m-d') : '';
                     }
 
                     if ($request->filled('checkout_date')) {

app/Http/Transformers/ActionlogsTransformer.php

@@ -113,7 +113,7 @@ class ActionlogsTransformer
             ] : null,
 
             'note'          => ($actionlog->note) ? e($actionlog->note): null,
-            'signature_file'   => ($actionlog->signature_filename) ? route('log.signature.view', ['filename' => $actionlog->signature_filename ]) : null,
+            'signature_file'   => ($actionlog->accept_signature) ? route('log.signature.view', ['filename' => $actionlog->accept_signature ]) : null,
             'log_meta'          => ((isset($clean_meta)) && (is_array($clean_meta))) ? $clean_meta: null,
             'action_date'   => ($actionlog->action_date) ? Helper::getFormattedDateObject($actionlog->action_date, 'datetime'): null,
 

config/version.php

@@ -1,10 +1,10 @@
 <?php
 return array (
-  'app_version' => 'v5.0.1',
-  'full_app_version' => 'v5.0.1 - build 5383-g870b097f0',
-  'build_version' => '5383',
+  'app_version' => 'v5.0.2',
+  'full_app_version' => 'v5.0.2 - build 5414-gc3e8f6406',
+  'build_version' => '5414',
   'prerelease_version' => '',
-  'hash_version' => 'g870b097f0',
-  'full_hash' => 'v5.0.0-1-g870b097f0',
+  'hash_version' => 'gc3e8f6406',
+  'full_hash' => 'v5.0.1-19-gc3e8f6406',
   'branch' => 'master',
 );

resources/assets/less/skins/skin-green-dark.less

@@ -144,6 +144,10 @@ a, a:link, a:visited, .btn-primary.hover {
 #assetsListingTable>tbody>tr.selected>td {
   background-color: var(--back-main);
 }
+#assetsListingTable>tbody>tr>td {
+  color: var(--link);
+}
+
 body {
   color: var(--text-main);
 }
@@ -201,6 +205,13 @@ body {
   background-color: var(--back-main);
   color: var(--text-main);
 }
+.dynamic-form-row{
+  color: @green;
+}
+
+.select2-container--default .select2-selection--single .select2-selection__rendered {
+  color: var(--header);
+}
 .form-control {
   background-color: var(--back-main);
   color: var(--text-main);
@@ -219,6 +230,9 @@ input[type=text], input[type=search] {
   background-color: var(--back-sub)!important;
   color: var(--text-main);
 }
+li.select2-results__option{
+  color:@green;
+}
 #licensesTable>tbody>tr>td>nobr>a>i.fa {
   color: var(--text-main);
 }
@@ -293,7 +307,9 @@ input[type=text], input[type=search] {
 #webui>div>div>div>div>div>table>tbody>tr>td>a>i.fa {
   color: var(--text-main);
 }
-
+#webui>#app>.row>.col-md-12>.box>.box-body>.row>.col-md-12 {
+  color:@green;
+}
 a {
   color: var(--link);
   &:link {
@@ -333,4 +349,3 @@ a {
   border-top: 1px solid #dddddd;
   display: table-cell;
 }
-

resources/lang/en/admin/settings/general.php

@@ -6,7 +6,7 @@ return array(
     'ad_domain_help'			=> 'This is sometimes the same as your email domain, but not always.',
     'ad_append_domain_label'    => 'Append domain name',
     'ad_append_domain'          => 'Append domain name to username field',
-    'ad_append_domain_help'     => 'User doesn\'t require to write "username@domain.local", they can just type "username".' ,
+    'ad_append_domain_help'     => 'User isn\'t required to write "username@domain.local", they can just type "username".' ,
     'admin_cc_email'            => 'CC Email',
     'admin_cc_email_help'       => 'If you would like to send a copy of checkin/checkout emails that are sent to users to an additional email account, enter it here. Otherwise leave this field blank.',
     'is_ad'				        => 'This is an Active Directory server',

resources/views/account/accept/create.blade.php

@@ -11,7 +11,7 @@
 @section('content')
 
     
-<link rel="stylesheet" href="{{ mix('css/signature-pad.min.css') }}">
+<link rel="stylesheet" href="{{ url('css/signature-pad.min.css') }}">
 
 <style>
 .form-horizontal .control-label, .form-horizontal .radio, .form-horizontal .checkbox, .form-horizontal .radio-inline, .form-horizontal .checkbox-inline {
@@ -25,6 +25,12 @@
     overflow: scroll;
 }
 
+.m-signature-pad--body {
+    border-style: solid;
+    border-color: grey;
+    border-width: thin;
+}
+
 </style>
 
 

resources/views/hardware/requested.blade.php

@@ -39,7 +39,6 @@
                     data-pagination="true"
                     data-id-table="requestedAssets"
                     data-cookie-id-table="requestedAssets"
-                    data-url="{{ route('api.consumables.index') }}"
                     data-export-options='{
                     "fileName": "export-assetrequests-{{ date('Y-m-d') }}",
                     "ignoreColumn": ["actions","image","change","checkbox","checkincheckout","icon"]

resources/views/users/view.blade.php

@@ -353,6 +353,8 @@
                       <form action="{{ route('users/bulkedit') }}" method="POST">
                         <!-- CSRF Token -->
                         <input type="hidden" name="_token" value="{{ csrf_token() }}" />
+                        <input type="hidden" name="bulk_actions" value="delete" />
+
                         <input type="hidden" name="ids[{{ $user->id }}]" value="{{ $user->id }}" />
                         <button style="width: 100%;" class="btn btn-sm btn-danger hidden-print">{{ trans('button.checkin_and_delete') }}</button>
                       </form>