Dropzone form post edited

akaunting · May 26, 2021 · 93ba76e · 93ba76e
1 parent 7adb8c9
commit 93ba76e
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 49 deletions.
diff --git a/app/Http/Controllers/Settings/Settings.php b/app/Http/Controllers/Settings/Settings.php
@@ -16,7 +16,7 @@ class Settings extends Controller
 {
     use DateTime, Uploads;
 
-    public $skip_keys = ['company_id', '_method', '_token', '_prefix'];
+    public $skip_keys = ['company_id', '_method', '_token', '_prefix', 'uploaded_logo'];
 
     public $file_keys = ['company.logo', 'invoice.logo'];
 
@@ -113,10 +113,6 @@ public function update(Request $request)
             }
 
             if ($real_key == 'default.locale') {
-                if (!in_array($value, config('language.allowed'))) {
-                    continue;
-                }
-
                 user()->setAttribute('locale', $value)->save();
             }
 
@@ -160,10 +156,10 @@ protected function oneCompany($real_key, $value)
                 Installer::updateEnv(['MAIL_FROM_NAME' => '"' . $value . '"']);
                 break;
             case 'company.email':
-                Installer::updateEnv(['MAIL_FROM_ADDRESS' => '"' . $value . '"']);
+                Installer::updateEnv(['MAIL_FROM_ADDRESS' => $value]);
                 break;
             case 'default.locale':
-                Installer::updateEnv(['APP_LOCALE' => '"' . $value . '"']);
+                Installer::updateEnv(['APP_LOCALE' => $value]);
                 break;
             case 'schedule.time':
                 Installer::updateEnv(['APP_SCHEDULE_TIME' => '"' . $value . '"']);

diff --git a/app/Http/Controllers/Wizard/Companies.php b/app/Http/Controllers/Wizard/Companies.php
@@ -50,7 +50,7 @@ public function update(Request $request)
 
         $fields = $request->all();
 
-        $skip_keys = ['company_id', '_method', '_token'];
+        $skip_keys = ['company_id', '_method', '_token', 'uploaded_logo'];
         $file_keys = ['company.logo'];
 
         foreach ($fields as $key => $value) {
@@ -70,13 +70,13 @@ public function update(Request $request)
                     $real_key = 'company.' . $key;
             }
 
-            // Process file uploads
-            if (in_array($real_key, $file_keys)) {
+             // Process file uploads
+             if (in_array($real_key, $file_keys)) {
                 // Upload attachment
                 if ($request->file($key)) {
                     $media = $this->getMedia($request->file($key), 'settings');
 
-                    $company->attachMedia($media, Str::snake($key));
+                    $company->attachMedia($media, Str::snake($real_key));
 
                     $value = $media->id;
                 }
@@ -87,7 +87,7 @@ public function update(Request $request)
                 }
             }
 
-            setting()->set($real_key, $value);
+            setting()->set($real_key, $value);            
         }
 
         // Save all settings

diff --git a/resources/assets/js/mixins/spa-global.js b/resources/assets/js/mixins/spa-global.js
@@ -98,7 +98,7 @@ export default {
                     file = this.$refs.dropzoneWizard.files[0];
                 }
             }
-
+            
             if(plus_data == 'logo') {
                 Object.assign(data, {
                     ['logo']: file

diff --git a/resources/assets/js/views/wizard/Company.vue b/resources/assets/js/views/wizard/Company.vue
@@ -70,12 +70,12 @@
             </div>
             <div class="col-6">
               <base-input :label="translations.company.logo">
-                  <akaunting-dropzone-file-upload
-                    ref="dropzoneWizard"
-                    preview-classes="single"
-                    :attachments="logo"
-                  >
-                  </akaunting-dropzone-file-upload>
+                <akaunting-dropzone-file-upload
+                  ref="dropzoneWizard"
+                  preview-classes="single"
+                  :attachments="logo"
+                >
+                </akaunting-dropzone-file-upload>
               </base-input>
             </div>
           </div>
@@ -127,52 +127,44 @@ export default {
     return {
       active: 0,
       logo: [],
-      real_date: ''
-      };
+      real_date: "",
+    };
   },
   mounted() {
+    let company_data = this.company;
     setTimeout(() => {
-      if (this.company != undefined) {
-        let logo_arr = [
-          {
-            id: this.company.logo.id,
-            name:this.company.logo.filename + "." + this.company.logo.extension,
-            path: this.company.logo.path,
-            type: this.company.logo.mime_type,
-            size: this.company.logo.size,
-            downloadPath: false,
-          },
-        ];
-        this.logo.push(logo_arr);
-        this.real_date = this.company.financial_start;
-      }
+      this.dataWatch(company_data);
     }, 500);
   },
   watch: {
     company: function (company) {
-      let logo_arr = [
-        {
-          id: company.logo.id,
-          name: company.logo.filename + "." + company.logo.extension,
-          path: company.logo.path,
-          type: company.logo.mime_type,
-          size: company.logo.size,
-          downloadPath: false,
-        },
+      this.dataWatch(company);
+    },
+  },
+  methods: {
+    dataWatch(company) {
+      if(Object.keys(company).length) {
+         let logo_arr = [
+            {
+              id: company.logo.id,
+              name: company.logo.filename + "." + company.logo.extension,
+              path: company.logo.path,
+              type: company.logo.mime_type,
+              size: company.logo.size,
+              downloadPath: false,
+            },
       ];
       this.logo.push(logo_arr);
-
       this.real_date = company.financial_start;
+      }
     },
-  },
-  methods: {
     next() {
       if (this.active++ > 2);
       this.$router.push("/wizard/currencies");
     },
 
     onEditSave() {
-      this.onEditEvent("PATCH", url + "/wizard/company", "logo", "", "");
+      this.onEditEvent("PATCH", url + "/wizard/companies", "logo", "", "");
       this.$router.push("/wizard/currencies");
     },
   },

diff --git a/routes/wizard.php b/routes/wizard.php
@@ -12,7 +12,7 @@
     Route::get('data', 'Wizard\Data@index')->name('data.index');
 
     Route::get('companies', 'Wizard\Companies@edit')->name('companies.edit');
-    Route::patch('companies', 'Wizard\Companies@update')->name('companies.update');
+    Route::patch('companies', 'Wizard\Companies@update')->middleware('dropzone')->name('companies.update');
 
     Route::get('currencies/{currency}/enable', 'Settings\Currencies@enable')->name('currencies.enable');
     Route::get('currencies/{currency}/disable', 'Settings\Currencies@disable')->name('currencies.disable');