search

callmemagnus / nextcloud-searchpage

App for Nextcloud providing a proper search page
7 stars 3 forks source link

[BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) #37

Closed JosefAschauer closed 2 months ago

JosefAschauer commented 3 months ago

Describe the bug A clear and concise description of what the bug is.

clicking the serch icon, which gave me wunderful search experience in the past, does simply nothing now but reloading the current nextcloud page - no search window appears anymore

Instance: AIO, nextcloud 19.03, behavior started back in 18.x uninstalling / removing and reinstalling does not help

callmemagnus commented 3 months ago

Hello,

This is strange. I'm using docker Nextcloud 28 and upgraded recently to 29 without issues. But not the AIO version.

The click on the app icon should be handled by Nextcloud. The icon is just a link on my instances. Clicking on the link redirects the user to this application page (/apps/thesearchpage/).

How do you install this application ? Manually or from the /settings/apps ?

Can you check that the URL is correct on the icon ? image

If you send me a HAR file of what is happening when you click and it reloads the page, it would help me spot if the issue is on the application.

Do you have plugins that changes the way the menu is presented ?

Lastly, is there any errors in the logs of Nextcloud ?

Thanks in advance

JosefAschauer commented 3 months ago

Hi,

Thanks for picking this up! The link on the icon points to my nextcloud URL is changing, dependent on where I am in the navigation (ie /settings/apps/search when I’, in /settings/apps, /apps/files/files if I click it in the files menu. This is apparently wrong...how do I fix it?

Please find the har file here: https://nc.abraxas.dyn-dns.org/s/FKrrA9oqW5yQcMn

The app was installed via settings/apps I have no app installed which changes the layout Icon seems correct No errors in the log (just OCR related from tesseract)

Best, Josef

From: magnus anderssen @.**@.>> Sent: Tuesday, 2 July 2024 09:49 To: callmemagnus/nextcloud-searchpage @.**@.>> Cc: Josef Aschauer @.**@.>>; Author @.**@.>> Subject: Re: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Hello,

This is strange. I'm using docker Nextcloud 28 and upgraded recently to 29 without issues. But not the AIO version.

The click on the app icon should be handled by Nextcloud. The icon is just a link on my instances. Clicking on the link redirects the user to this application page (/apps/thesearchpage/).

How do you install this application ? Manually or from the /settings/apps ?

Can you check that the URL is correct on the icon ? image.png (view on web)https://github.com/callmemagnus/nextcloud-searchpage/assets/232478/81941ad4-9200-4880-91ea-a7478b3c2d3c

If you send me a HAR file of what is happening when you click and it reloads the page, it would help me spot if the issue is on the application.

Do you have plugins that changes the way the menu is presented ?

Lastly, is there any errors in the logs of Nextcloud ?

Thanks in advance

— Reply to this email directly, view it on GitHubhttps://github.com/callmemagnus/nextcloud-searchpage/issues/37#issuecomment-2202212630, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBOXNFT7GBBGBY4HJ6U73DZKJLPVAVCNFSM6AAAAABKFUB4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBSGIYTENRTGA. You are receiving this because you authored the thread.Message ID: @.**@.>>

JosefAschauer commented 3 months ago

PS: if I deliberately enter /apps/thesearchpage in the browser, thesearchpage opens just fine. Seems the icon is really just missing a proper link.

Seems fulltextseach is also having some hickups since some time. As soon as I click on a file in the result list, it just opens the folder with the file, underlaying action is &scrollto

From: Josef Aschauer Sent: Tuesday, 2 July 2024 18:31 To: 'callmemagnus/nextcloud-searchpage' @.>; 'callmemagnus/nextcloud-searchpage' @.> Cc: 'Author' @.***> Subject: RE: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Hi,

Thanks for picking this up! The link on the icon points to my nextcloud URL is changing, dependent on where I am in the navigation (ie /settings/apps/search when I’, in /settings/apps, /apps/files/files if I click it in the files menu. This is apparently wrong...how do I fix it?

Please find the har file here: https://redacted/s/FKrrA9oqW5yQcMn

The app was installed via settings/apps I have no app installed which changes the layout Icon seems correct No errors in the log (just OCR related from tesseract)

Best, Josef

From: magnus anderssen @.**@.>> Sent: Tuesday, 2 July 2024 09:49 To: callmemagnus/nextcloud-searchpage @.**@.>> Cc: Josef Aschauer @.**@.>>; Author @.**@.>> Subject: Re: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Hello,

This is strange. I'm using docker Nextcloud 28 and upgraded recently to 29 without issues. But not the AIO version.

The click on the app icon should be handled by Nextcloud. The icon is just a link on my instances. Clicking on the link redirects the user to this application page (/apps/thesearchpage/).

How do you install this application ? Manually or from the /settings/apps ?

Can you check that the URL is correct on the icon ? image.png (view on web)https://github.com/callmemagnus/nextcloud-searchpage/assets/232478/81941ad4-9200-4880-91ea-a7478b3c2d3c

If you send me a HAR file of what is happening when you click and it reloads the page, it would help me spot if the issue is on the application.

Do you have plugins that changes the way the menu is presented ?

Lastly, is there any errors in the logs of Nextcloud ?

Thanks in advance

— Reply to this email directly, view it on GitHubhttps://github.com/callmemagnus/nextcloud-searchpage/issues/37#issuecomment-2202212630, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBOXNFT7GBBGBY4HJ6U73DZKJLPVAVCNFSM6AAAAABKFUB4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBSGIYTENRTGA. You are receiving this because you authored the thread.Message ID: @.**@.>>

callmemagnus commented 3 months ago

Oddly enough, I have now the same issues, but not always (?) on this application but several others too.

I filed a bug upstream to get their recommendations.

JosefAschauer commented 3 months ago

Hi, Seems the bug I reported for thesearchpage not loading is also caused by fulltextsearch. Once you disable the navigation icon, all works as expected.

@.***

Found here: https://github.com/nextcloud/fulltextsearch/issues/842

Best, Josef

From: magnus anderssen @.> Sent: Wednesday, 3 July 2024 09:19 To: callmemagnus/nextcloud-searchpage @.> Cc: Josef Aschauer @.>; Assign @.> Subject: Re: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Oddly enough, I have now the same issues, but not always (?) on this application but several others too.

I filed a bug upstream to get their recommendations.

— Reply to this email directly, view it on GitHubhttps://github.com/callmemagnus/nextcloud-searchpage/issues/37#issuecomment-2205269025, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBOXNC7XDSFOIUQIPD2N6TZKOQXTAVCNFSM6AAAAABKFUB4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBVGI3DSMBSGU. You are receiving this because you were assigned.Message ID: @.**@.>>

JosefAschauer commented 3 months ago

Hi, could you please remove my url from my last email?

Gesendet von Outlook für Androidhttps://aka.ms/AAb9ysg

From: magnus anderssen @.> Sent: Tuesday, July 2, 2024 3:34:53 PM To: callmemagnus/nextcloud-searchpage @.> Cc: Josh @.>; Assign @.> Subject: Re: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Assigned #37https://github.com/callmemagnus/nextcloud-searchpage/issues/37 to @JosefAschauerhttps://github.com/JosefAschauer.

— Reply to this email directly, view it on GitHubhttps://github.com/callmemagnus/nextcloud-searchpage/issues/37#event-13366303484, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBOXNDBG54PLYCH4FJZDXLZKKT73AVCNFSM6AAAAABKFUB4YSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGM3DMMZQGM2DQNA. You are receiving this because you were assigned.Message ID: @.***>

callmemagnus commented 3 months ago

I redacted your comment but you can do it yourself through github's interface.

JosefAschauer commented 3 months ago

many thanks callmemagnus, haven't had my github login with me while on mobile.

cociweb commented 2 months ago

Hi, As a workaround (which is recommended in our case), please disable the navigation icon of the original FullTextSearch As recommended here: https://github.com/nextcloud/fulltextsearch/issues/842#issuecomment-2139337918

Actually it is not a workaround (This is the solution!), since the Bug is in FullTextSearch, so we can close the thread if it is confirmed. (But better to pin it to avoid the duplicates.)

JosefAschauer commented 2 months ago

Thanks mate, already done werks ago;)

Gesendet von Outlook für Androidhttps://aka.ms/AAb9ysg

From: cociweb @.> Sent: Wednesday, July 31, 2024 1:08:40 PM To: callmemagnus/nextcloud-searchpage @.> Cc: Josh @.>; Mention @.> Subject: Re: [callmemagnus/nextcloud-searchpage] [BUG] the search page does not work anymore since Nextcloud 28.x (maybe 29) (Issue #37)

Hi, As a workaround (which is recommended in our case), please disable the navigation icon of the original Full Text Search. As recommended here: nextcloud/fulltextsearch#842 (comment)https://github.com/nextcloud/fulltextsearch/issues/842#issuecomment-2139337918

— Reply to this email directly, view it on GitHubhttps://github.com/callmemagnus/nextcloud-searchpage/issues/37#issuecomment-2260264970, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBOXNHUXR5SGLPX25I6HDLZPDATRAVCNFSM6AAAAABKFUB4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRQGI3DIOJXGA. You are receiving this because you were mentioned.Message ID: @.***>

callmemagnus commented 2 months ago

I close this bug as it's outside of the scope of this application.