Giter Club home page Giter Club logo

Comments (19)

JameZUK avatar JameZUK commented on June 26, 2024 3

@prolibre If I run it manually I get that error too. But as you said, the error in the logs has gone away. I'm certainly not a NC dev so this may not actually be a fix.. If anyone knows how you can test this properly, I'm happy to give it a go.

You can see the difference here: master...JameZUK:fulltextsearch:master

from fulltextsearch.

prolibre avatar prolibre commented on June 26, 2024 2

@JameZUK I've restarted the application using your script and I'm taking the opportunity to restart the indexing. I'm going to keep a close eye on this to see if there are any other errors. Thanks again for your help.

from fulltextsearch.

derritter88 avatar derritter88 commented on June 26, 2024 1

@joshtrichards applied your patch as well - same issue as mentioned by @prolibre for me.

from fulltextsearch.

EVOTk avatar EVOTk commented on June 26, 2024 1

Hello,
Same issue for me with NC 29

from fulltextsearch.

JameZUK avatar JameZUK commented on June 26, 2024 1

I managed to get the error to go away and I think its working for me now. Uploaded here: https://github.com/JameZUK/fulltextsearch/tree/master/lib/Cron

from fulltextsearch.

kmorber avatar kmorber commented on June 26, 2024

Nextcloud 29.0 beta 3

from fulltextsearch.

stefan-niedermann avatar stefan-niedermann commented on June 26, 2024

Apparently the fulltextsearch app

need[s] to migrate away from OC\BackgroundJob\TimedJob towards OCP\BackgroundJob\TimedJob before releasing for 29 (Source)

from fulltextsearch.

joshtrichards avatar joshtrichards commented on June 26, 2024

Feel free to test the proposed fix in #829 if you are so inclined and report back.

from fulltextsearch.

prolibre avatar prolibre commented on June 26, 2024

Hello @joshtrichards

I've done the test by installing the two files from #829 (Index.php and Maintenance.php) on my server (version 29.0.0.19) but I get an error when executing the cron

ArgumentCountError: Too few arguments to function OCA\FullTextSearch\Cron\Index::__construct(), 0 passed in /(...)/nextcloud/lib/private/BackgroundJob/JobList.php on line 330 and exactly 7 expected in /(...)/nextcloud/apps/fulltextsearch/lib/Cron/Index.php:63

Thanks a lot

from fulltextsearch.

JameZUK avatar JameZUK commented on June 26, 2024

Same here for NC 29 and the fix provided by @joshtrichards

"Too few arguments to function OCA\FullTextSearch\Cron\Index::__construct(), 0 passed in /var/www/html/lib/private/BackgroundJob/JobList.php on line 330 and exactly 7 expected"

from fulltextsearch.

kmorber avatar kmorber commented on June 26, 2024

Same here on NC 29.0

from fulltextsearch.

DantesPop avatar DantesPop commented on June 26, 2024

I am getting the same error running Nextcloud 29.0.0.19
Error: Class "OC\BackgroundJob\TimedJob" not found in /var/www/nextcloud/apps/fulltextsearch/lib/Cron/Index.php:51

from fulltextsearch.

prolibre avatar prolibre commented on June 26, 2024

I managed to get the error to go away and I think its working for me now. Uploaded here: https://github.com/JameZUK/fulltextsearch/tree/master/lib/Cron

Thanks a lot @JameZUK . However, I've just installed your files and I'm getting an error when I run the cron :-(

Error: Typed property OCP\BackgroundJob\Job::$time must not be accessed before initialization in /(...)/nextcloud/lib/public/BackgroundJob/TimedJob.php:101
Stack trace:
#0 /(...)/nextcloud/lib/public/BackgroundJob/TimedJob.php(92): OCP\BackgroundJob\TimedJob->start()
#1 /(...)/nextcloud/cron.php(176): OCP\BackgroundJob\TimedJob->execute()
#2 {main}

EDIT: I'm not actually sure that the problem is with your script. When I restart the cron I no longer get the error and I can't see anything linking the error to fulltextsearch :-)

from fulltextsearch.

D3nnis3n avatar D3nnis3n commented on June 26, 2024

Still having this on release version. Seems to be a lot of problems with fulltextsearch and plugins - some don't even update.

from fulltextsearch.

AInteriorB avatar AInteriorB commented on June 26, 2024

can confirm the issue

from fulltextsearch.

aakerbeere avatar aakerbeere commented on June 26, 2024

I also get this with the release version (29.0.0.19). I actually disabled "fulltextsearch: 28.0.1". Cron runs as expected.

from fulltextsearch.

AInteriorB avatar AInteriorB commented on June 26, 2024

Quick workaround: I just added a P in the namesprace
use OCP\BackgroundJob\TimedJob;
instead of
use OC\BackgroundJob\TimedJob;

in two files

lib/Cron/Index.php:53
lib/Cron/Maintenance.php:43

After first run (that throws the mentioned error #828 (comment)), everything works good

from fulltextsearch.

D3nnis3n avatar D3nnis3n commented on June 26, 2024

Doesn't work for me, I get the error on every subsequent execution as well.

from fulltextsearch.

joshtrichards avatar joshtrichards commented on June 26, 2024

Fixed in #836 / #838

from fulltextsearch.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.