Bug #11794
closed
If website root is active, searching for links does not work
100%
Description
When the globe (page id = 0) is active, searching for broken links does not display any result.
This was working once - at least there is a picture in the manual showing the working module while the rootpage is activated.
Either display a hint to choose a page first (as the page module does) or make it work again (would be even better).
Updated by Chris topher almost 14 years ago
- Status changed from Resolved to Needs Feedback
Hmm, does not work for me. :-(
What should happen:
When the rootpage is activated, the statistics table and the table showing the broken links should work just as for every other page. That means, depending on the search_level set, they should display the number of results (in the stats table) and the broken links for the checked options (in the second table).
But currently when I activate the rootpage, the statistics table always displays zero results and there also are none for the broken links table.
Updated by Pierre Boivin almost 14 years ago
I just roll back my solution.
Seems like, it doesn't work anyway for Depth selection.
The problem is "t3lib_tsfeBeUserAuth::extGetTreeList" that is use and doesn't support page id "0".
I think we should display a message instead.
Updated by Pierre Boivin almost 14 years ago
I added a warning message in case the rootpage (zero) is activated.
Updated by Chris topher almost 14 years ago
Much better than before!
But it has the side-effect that you can no longer check broken pages for the whole tree on one screen by selecting the globe and some search depth. If you have e.g. 15 subpages under the globe you now have to check each of them seperately.
Looking at usability it would be better, if it worked again with id=0; do you think you could try to fix it that way?
Updated by Pierre Boivin almost 14 years ago
- Assignee deleted (
Pierre Boivin)
I can try but probably not this week.
As I said, we will have to use another function than "t3lib_tsfeBeUserAuth::extGetTreeList" because it doesn't support id=0
Updated by Pierre Boivin almost 14 years ago
- Assignee deleted (
Pierre Boivin)
Corrected.
Those modifications needs test before closing the issue.
Updated by Pierre Boivin almost 14 years ago
- Status changed from Needs Feedback to Resolved
- % Done changed from 50 to 100
Applied in changeset r41875.
Updated by Chris topher almost 14 years ago
- Status changed from Resolved to Under Review
Wow, thank you for fixing this a second time!
I think when I wrote the bug report I should immediatelly have said more clearly, which solution I really wanted - sorry for the inconveniences!
I will review it today!
Updated by Chris topher almost 14 years ago
- Status changed from Under Review to Needs Feedback
The layout of the backend interface is working correctly.
But the logic to get $pageList does not work correctly:- If I have a page with another id than 0 selected, I get the same broken links like before.
- With pid=0 I get no results. Reason seems to be that pageList in that case stays empty.
Updated by Pierre Boivin almost 14 years ago
I can't reproduce this bug. It works here for me.
Updated by Pierre Boivin almost 14 years ago
- Status changed from Needs Feedback to Under Review
You can test again. Probably it was caused by shortcut pages or other page type.
Updated by Philipp Gampe almost 14 years ago
works for me ... make sure to select the the amount of subpages to search for !!! current page will not show any result ;)
Updated by Chris topher almost 14 years ago
As far as I can tell, Pierre's patch is working. Sure you have to set the level of pages to search to something different than "this page", but that is only logical.
Btw @ Pierre: Good guess with the shortcut pages - that was the problem. :-)
I'd say this one is solved.
Updated by Chris topher almost 14 years ago
- Status changed from Under Review to Resolved
Updated by Chris topher over 12 years ago
- Status changed from Resolved to Closed
Updated by Michael Stucki almost 11 years ago
- Project changed from 1510 to TYPO3 Core
- Category changed from Linkvalidator to Linkvalidator