Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation failing with postgres #9783

Closed
1 task done
DrjavaB opened this issue May 10, 2024 · 4 comments
Closed
1 task done

Installation failing with postgres #9783

DrjavaB opened this issue May 10, 2024 · 4 comments
Labels
Feature Request Issues filed as feature requests.

Comments

@DrjavaB
Copy link

DrjavaB commented May 10, 2024

Terms

  • Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected

2.1.2

Issue Description

2023_05_26_213105_create_wishlist_items_table ................................. 16ms DONE
2023_05_26_213120_create_compare_items_table ................................... 9ms DONE
2023_06_27_163529_rename_product_review_images_to_product_review_attachments .. 17ms DONE
2023_07_06_140013_add_logo_path_column_to_locales .............................. 1ms FAIL

In Connection.php line 822:

  SQLSTATE[42703]: Undefined column: 7 ERROR:  column "locales/" does not exist             
  LINE 1: update "locales" set "logo_path" = CONCAT("locales/", code, ...                   
                                                    ^ (Connection: pgsql, SQL: update "loc  
  ales" set "logo_path" = CONCAT("locales/", code, ".png") where "logo_path" is null)   

In Connection.php line 605:

  SQLSTATE[42703]: Undefined column: 7 ERROR:  column "locales/" does not exist  
  LINE 1: update "locales" set "logo_path" = CONCAT("locales/", code, ...        
    
```                                                ^                            
                                                                                 



### Preconditions

fresh postgres installation and bagiso latest version.
installation process using cli (`php artisan migrate`) not gui

@DrjavaB DrjavaB added the Bug Something isn't working. label May 10, 2024
@rahulthomasdev
Copy link

rahulthomasdev commented May 11, 2024

Same issue here. The main problem is in the migration and seed files as it toilered to work with mysql it seems and not with psql, from few errors i confirmed that we need to modify each to work with psql. Now talking about this specific issue, change the line in bagista\bagisto\packages\Webkul\Core\src\Database\Migrations\2023_07_06_140013_add_logo_path_column_to_locales.php to DB::table('locales')->whereNull('logo_path')->update(['logo_path' => DB::raw('CONCAT(\'locales/\', code, \'.png\')')]);

@rahulthomasdev
Copy link

I'm encountering difficulties with other migrations too. It seems like the codebase is heavily dependent on SQL/MySQL.

@devansh-webkul
Copy link
Member

We dont have postgress support as of now.

@devansh-webkul
Copy link
Member

In initial phase our migration files are highly dependent on the mysql files and however postgress is also in our roadmap but this feature is quite time consuming because we need to plan it for existing as well as new users.

@devansh-webkul devansh-webkul added Feature Request Issues filed as feature requests. and removed Bug Something isn't working. labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Issues filed as feature requests.
Projects
None yet
Development

No branches or pull requests

4 participants