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

Cant find dbtableprefix value in cloud config #28

Closed
xkill opened this issue Nov 25, 2021 · 4 comments · Fixed by #34
Closed

Cant find dbtableprefix value in cloud config #28

xkill opened this issue Nov 25, 2021 · 4 comments · Fixed by #34
Labels
bug Something isn't working Fixed for upcoming version Fixed in new version

Comments

@xkill
Copy link

xkill commented Nov 25, 2021

Describe the bug
After performing the automated installation of deps I got the error:

Cant find `dbtableprefix` value in cloud config

To Reproduce
Steps to reproduce the behavior:

  1. Install plugin
  2. Go to MediaDC preferences
  3. Click on "Check installation"
  4. See error

Expected behavior
If the dbtableprefix is not defined get this value from default settings. In this case this value is oc_

Desktop (please complete the following information):

  • Nextcloud 22.2.3
  • Database configuration. using MariaDB with default dbtableprefix (oc_)
  • Python 3.8.10
  • MediaDC 0.1.7
@bigcat88
Copy link
Member

Do you have in your config file located at NEXTCLOUD_FOLDER/config/config.php value dbtableprefix ?
https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/config_sample_php_parameters.html?highlight=dbtableprefix

@xkill
Copy link
Author

xkill commented Nov 26, 2021

This is the work around: To manually add the dbtableprefix to the config.php, but the application should get the default value (if it is not defined) in order to work fine.

@bigcat88
Copy link
Member

But it helps? Or problem is somewhere else?
I agree with you that it is a workaround, you are right, if it works when you add dbtableprefix then we know where to fix it, and will fix it.
Next update(with fix for shared directories scan and GUI improvements) planned in close future, with a couple of days.

@bigcat88 bigcat88 added bug Something isn't working Fixed for upcoming version Fixed in new version labels Nov 29, 2021
bigcat88 added a commit that referenced this issue Nov 29, 2021
@andrey18106 andrey18106 linked a pull request Dec 20, 2021 that will close this issue
@bigcat88
Copy link
Member

@xkill Fixed in 0.1.8

andrey18106 added a commit that referenced this issue Oct 28, 2022
* Funding update

* Fixed: user shared, webdav shares, local shares files scan.

* Fix for #28

* Update pylint config file.

* MediaDC 0.1.8 features. Batch editing. Bug fixes

* Removed npm audit from required for check

* Increased supported Nextcloud version

* Installation check fix

* Screenshots update.

* Added new screenshot.

* Minor fixes

* Minor changes

* Fixed deselection

* Removed debug stuff.

* Bug fixes. Added option auto opening of the next group #30

* Error messages output fixes

* Minor changes

* Added output to errors

* Psalm fix

* Update description of python_command.

* Some bug fixes

* Minor changes on error messages

* Removed log

* Changed boost package availability

* Output messages fixes

* User rights changes

* MediaDC 0.1.8 changes

* Removed documentation label from stale workflow

* API permissions update

* Psalm fix

* Updated changelog

* Support info update

* Updated screenshots

* Changed make command for production build

* null error bug fix

* Postgre slash fix. #44

* now no clearing dbhost value, `usock` var always have a higher priority in `create_connection. #45

* this probably will fix that situation #47

* pylint: max-returns from 6 to 8

* #47 addition check not to save to table too large values.

* #48
remove version check, now check only if we see such tools.

* Minor changes

* Added task finish notification. Minor refactoring and fixes

* mediadc:collector:tasks:notify

* mediadc:collector:tasks:notify
+collector_settings['finish_notification']

* max-branches from 12 -> 14

* small adjustment if task_info does not contain those fields.

* Added missed setting

* f%ng php cli. =)

* Fixed typo

* Updated bug report info

* Using new pillow_heif 0.1.6 functions.

* Minor fixes

* MediaDC 0.1.9 changelog

* Workflow update

* Revert changes back, till version 0.2.0

* Sync branch dev with main

* possible fix for python 3.6 EOL get-pip.py

* added curl redirect handling.

* added logs to terminal.

* actions/checkout to v3.

* fix for logs.

* Added notice. Added CoD. Turned off cron schedule for stale workflow

* Bug report workflow update

* apply black

* apply isort

* move pylint config to pyproject.toml

* fix for deprecated `pillow_heif` functions.

* using pillow_heif as plugin.

* fix for image orientation tags.

* removed unused `path` parameter.

* bump ncipollo/release-action to v1.10.0 [ci skip]

* Python part for #61

* MediaDC old changes, refactoring, adding some new features, minor bug fixes

* eslint fixes

* Added Transifex config

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* adding `* text eol=lf`

* fix for python for #61

* l10n renamed plural translate function name

* fix for #61

* l10n: Fixes (#76)

* l10n: Fixed typo

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Changed casing

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Fixed typo and changed grammar

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Changed grammar

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* [tx-robot] Update transifex configuration

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* l10n plural function fix

* Added filesplugin, context menu option to scan folder

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Added ignored flag message

* Lint fixes

* Updated Transifex link

* Updated ocp versions

* Updated screenshots

* Added MediaDC 0.1.10 changelog

* Fixed files attributes

* Fixed attributes

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Updated date

* Fixed call to renamed function

Signed-off-by: Nextcloud bot <bot@nextcloud.com>
Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
Co-authored-by: Alexander Piskun <bigcat88@icloud.com>
Co-authored-by: Nextcloud bot <bot@nextcloud.com>
Co-authored-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
andrey18106 added a commit that referenced this issue Nov 16, 2022
* Funding update

* Fixed: user shared, webdav shares, local shares files scan.

* Fix for #28

* Update pylint config file.

* MediaDC 0.1.8 features. Batch editing. Bug fixes

* Removed npm audit from required for check

* Increased supported Nextcloud version

* Installation check fix

* Screenshots update.

* Added new screenshot.

* Minor fixes

* Minor changes

* Fixed deselection

* Removed debug stuff.

* Bug fixes. Added option auto opening of the next group #30

* Error messages output fixes

* Minor changes

* Added output to errors

* Psalm fix

* Update description of python_command.

* Some bug fixes

* Minor changes on error messages

* Removed log

* Changed boost package availability

* Output messages fixes

* User rights changes

* MediaDC 0.1.8 changes

* Removed documentation label from stale workflow

* API permissions update

* Psalm fix

* Updated changelog

* Support info update

* Updated screenshots

* Changed make command for production build

* null error bug fix

* Postgre slash fix. #44

* now no clearing dbhost value, `usock` var always have a higher priority in `create_connection. #45

* this probably will fix that situation #47

* pylint: max-returns from 6 to 8

* #47 addition check not to save to table too large values.

* #48
remove version check, now check only if we see such tools.

* Minor changes

* Added task finish notification. Minor refactoring and fixes

* mediadc:collector:tasks:notify

* mediadc:collector:tasks:notify
+collector_settings['finish_notification']

* max-branches from 12 -> 14

* small adjustment if task_info does not contain those fields.

* Added missed setting

* f%ng php cli. =)

* Fixed typo

* Updated bug report info

* Using new pillow_heif 0.1.6 functions.

* Minor fixes

* MediaDC 0.1.9 changelog

* Workflow update

* Revert changes back, till version 0.2.0

* Sync branch dev with main

* possible fix for python 3.6 EOL get-pip.py

* added curl redirect handling.

* added logs to terminal.

* actions/checkout to v3.

* fix for logs.

* Added notice. Added CoD. Turned off cron schedule for stale workflow

* Bug report workflow update

* apply black

* apply isort

* move pylint config to pyproject.toml

* fix for deprecated `pillow_heif` functions.

* using pillow_heif as plugin.

* fix for image orientation tags.

* removed unused `path` parameter.

* bump ncipollo/release-action to v1.10.0 [ci skip]

* Python part for #61

* MediaDC old changes, refactoring, adding some new features, minor bug fixes

* eslint fixes

* Added Transifex config

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* adding `* text eol=lf`

* fix for python for #61

* l10n renamed plural translate function name

* fix for #61

* l10n: Fixes (#76)

* l10n: Fixed typo

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Changed casing

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Fixed typo and changed grammar

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* l10n: Changed grammar

Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

* [tx-robot] Update transifex configuration

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* l10n plural function fix

* Added filesplugin, context menu option to scan folder

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Added ignored flag message

* Lint fixes

* Updated Transifex link

* Updated ocp versions

* Updated screenshots

* Added MediaDC 0.1.10 changelog

* Fixed files attributes

* Fixed attributes

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Updated date

* Fixed call to renamed function

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Updated code and dependencies for NC25 compatibility

Lint fixes

* Fixed typo (#80)

* Changed nextcloud ocp version

* Updated deps verions (for workflow fix)

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* MediaDC 0.2.0 changes.

Added new migration for tasks_details table changes

Code changes and refactoring according to the database changes

Moved initial data to the php file for L10N support

Lint fixes

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Psalm fixes

Added SERVER_ROOT env var for Python script

Minor vue fix

* Added ignore condition for tx-robot commits

* Audit fixes

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Minor fixes. Updated screenshots list

* Updated js binaries

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* [tx-robot] updated from transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

* Added 0.2.0 changelog

Signed-off-by: Nextcloud bot <bot@nextcloud.com>
Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
Co-authored-by: Alexander Piskun <bigcat88@icloud.com>
Co-authored-by: Nextcloud bot <bot@nextcloud.com>
Co-authored-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fixed for upcoming version Fixed in new version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants