search

nextcloud / weather

⛅️ Weather app for Nextcloud
GNU Affero General Public License v3.0
47 stars 27 forks source link

Column "oc_weather_config"."value" is type String, but exceeding the 4.000 length limit #107

Open Ayush62 opened 1 year ago

Ayush62 commented 1 year ago

Hello all,

I am trying to install the weather app on my nextcloud but i keep getting the same error.

I have my nextcloud installed on a raspberry pi 4 8GB ram with snap.

does someone know what to do?

ShangTianYa commented 1 year ago

fuck.....

ctorrean commented 1 year ago

Also seeing this issue, on both 25.0.2 and now on 25.0.3

mohataj commented 1 year ago

here too

brownje96 commented 1 year ago

Same issue.

JOHLC commented 1 year ago

Also here 

Error   settings    InvalidArgumentException: Column "oc_weather_config"."value" is type String, but exceeding the 4.000 length limit.      2023-01-28T23:10:52-0500
Error   settings    InvalidArgumentException: Column "oc_weather_config"."value" is type String, but exceeding the 4.000 length limit.      2023-01-28T23:10:23-0500
sergeng commented 1 year ago

Fix https://github.com/nextcloud/weather/pull/108/commits/00f1a33a33d6502739910ce52d3aca6592c97090

jonas740 commented 1 year ago

Fix 00f1a33

Ye so in a docker install of nextcloud how do you proceed to fix this ?

And i am on version Nextcloud Hub 3 25.0.4

lainedfles commented 1 year ago

Here is what worked for me:

  1. Modify the file to update the value to 4000: sudo docker exec --user www-data -it nextcloud-aio-nextcloud sed -Ee "s/'length'(\s+)=>(\s+)10240,/'length'\1=>\24000,/" -i.bak /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php
  2. Validate using diff: sudo docker exec --user www-data -it nextcloud-aio-nextcloud diff -uw /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php.bak /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php
  3. Navigate to yourdomain/settings/apps and re-enable the Weather application.
MrGenius1 commented 1 year ago

Here is what worked for me:

  1. Modify the file to update the value to 4000: sudo docker exec --user www-data -it nextcloud-aio-nextcloud sed -Ee "s/'length'(\s+)=>(\s+)10240,/'length'\1=>\24000,/" -i.bak /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php
  2. Validate using diff: sudo docker exec --user www-data -it nextcloud-aio-nextcloud diff -uw /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php.bak /var/www/html/custom_apps/weather/lib/Migration/Version010703Date20201101235744.php
  3. Navigate to yourdomain/settings/apps and re-enable the Weather application.

How to fix this on a Ubuntu install with Nextcloud Hub 3?

lainedfles commented 1 year ago

How to fix this on a Ubuntu install with Nextcloud Hub 3?

The steps outlined in my original reply are pretty basic. I'd recommend that you read about bash and Ubuntu in general. I don't run Ubuntu so I can only guess and provide a suggestion:

  1. Use find to locate Version010703Date20201101235744.php: sudo find / -iname 'Version010703Date20201101235744.php'
  2. Use the same sed command to apply the patch: sudo sed -Ee "s/'length'(\s+)=>(\s+)10240,/'length'\1=>\24000,/" -i.bak <FULLY_QUALIFIED_PATH_TO_Version010703Date20201101235744.php>
  3. Validate with diff: sudo diff -uw <FULLY_QUALIFIED_PATH_TO_Version010703Date20201101235744.php>.bak <FULLY_QUALIFIED_PATH_TO_Version010703Date20201101235744.php>
  4. Navigate to yourdomain/settings/apps and re-enable the Weather application

Alternative strategies:

ShangTianYa commented 1 year ago

Sorry, probably won't consider Nextcloud again due to poor optimization and poor performance, and of course thanks for your kind help :)

John Doe @.***> 于2023年3月14日周二 21:29写道：

How to fix this on a Ubuntu install with Nextcloud Hub 3?

The steps outlined in my original reply are pretty basic. I'd recommend that you read about bash and Ubuntu in general. I don't run Ubuntu so I can only guess and provide a suggestion:

  1. Use find to locate Version010703Date20201101235744.php: sudo find / -iname 'Version010703Date20201101235744.php'
  2. Use the same sed command to apply the patch: sudo sed -Ee "s/'length'(\s+)=>(\s+)10240,/'length'\1=>\24000,/" -i.bak
  3. Validate with diff: sudo diff -uw **.bak**
  4. Navigate to yourdomain/settings/apps and re-enable the Weather application

Alternative strategies:

— Reply to this email directly, view it on GitHub https://github.com/nextcloud/weather/issues/107#issuecomment-1469129178, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQQ6KQ5MICT2PZFPEPAGR5DW4ELO7ANCNFSM6AAAAAATY64E6E . You are receiving this because you commented.Message ID: @.***>

lainedfles commented 1 year ago

Sorry, probably won't consider Nextcloud again due to poor optimization and poor performance, and of course thanks for your kind help :)

I understand your frustration as optimization and performance are not universal although we've seen much progress over the years. I'm optimistic about the future of this project. I've yet to find any alternative close to feature parity. Many of the optimizations are now easier to configure using environment variables.

Tuning is a requirement IMO and worth the effort. It would be nice if more attention was dedicated to optimization & performance but I understand the effort required to maintain such an undertaking. For this reason I sympathize with the maintainers decision to abstain. P.s. thanks to all developers and contributors!

You're welcome. If only you've benefit, I'm satisfied.