Defect #34634
closed
Deletion of project wiki leaves the project wiki inaccessible (404) until module reactivation
Added by Ivan Cenov almost 4 years ago. Updated about 1 year ago.
0%
Description
This issue is inspired by a thread in the forum (Can't add deleted wiki Tab. (404))
An user delete all wiki pages including the root page. Then he could not see the root page and could not create wiki pages any more.
The solution was to disable Wiki module and then enable it again. Then the root wiki page appeared again (may be it has been recreated on enabling wiki module).
So I think that it should be harder to delete the root wiki page or make it not erasable.
However, if it remains erasable, Wiki module should be made inactive when this page is deleted.
Files
| 0004-Disable-the-wiki-module-after-deletion-of-a-project-.patch (2.21 KB) 0004-Disable-the-wiki-module-after-deletion-of-a-project-.patch | Marius BĂLTEANU, 2021-07-26 23:35 | ||
| 34634.patch (2.79 KB) 34634.patch | Marius BĂLTEANU, 2022-03-21 22:44 |
Related issues
Updated by Ivan Cenov almost 4 years ago
Environment
Redmine version 3.4.4.devel
Ruby version 2.4.1-p111 (2017-03-22) [x86_64-linux]
Rails version 5.1.4
Environment production
Database adapter Mysql2
Updated by Mischa The Evil almost 4 years ago
- Status changed from New to Confirmed
- Assignee set to Mischa The Evil
Thanks for opening this issue. I've been reading along and it already got me curious, so I already had a look at the source. I can confirm the reported behavior (at least partially, as far as I've tested). Along the way I found several other issues too.
I'm working on providing a patch.
Updated by Mischa The Evil almost 4 years ago
- Related to Patch #34643: Cleanups and fixes after Wiki tab removal from project settings (#26579) added
Updated by Mischa The Evil almost 4 years ago
- Category set to Wiki
- Assignee changed from Mischa The Evil to Ivan Cenov
Ivan Cenov wrote:
An user delete all wiki pages including the root page. Then he could not see the root page and could not create wiki pages any more.
Mischa The Evil wrote:
I can confirm the reported behavior (at least partially, as far as I've tested).
I confirm this is a problem when a whole project wiki is deleted. I don't see this being a problem when deleting single wiki pages.
Ivan Khrustalev: can you too confirm this (also with the OP of Can't add deleted wiki Tab. (404))?
I have posted a patch that fixes this case as part of #34643 (file: 4. Disable the wiki module after deletion of a project wiki.). It should be applicable on its own.
Updated by Marius BĂLTEANU over 3 years ago
- File 0004-Disable-the-wiki-module-after-deletion-of-a-project-.patch 0004-Disable-the-wiki-module-after-deletion-of-a-project-.patch added
- Assignee changed from Ivan Cenov to Mischa The Evil
- Target version set to 5.0.0
I've extracted the patch proposed by Mischa The Evil in #34643:
- 4. Disable the wiki module after deletion of a project wiki.
By adding a (private)after_destroy:disable_wiki_modulecallback that disables the wiki module after deletion of a project wiki, we prevent the need to explicitly have to disable the wiki module first, before (re-)enabling the module would eventually create a new project wiki and reset thestart_page.
I still think that it's a good ideea to inform the user that deleting the wiki root page will automatically disable the module.
Updated by Mischa The Evil over 3 years ago
Marius BALTEANU wrote:
I still think that it's a good ideea to inform the user that deleting the wiki root page will automatically disable the module.
Sure. However, now that I have had a second look at this, I think that disabling the module after the deletion is a bit of an ugly workaround. A better solution would be to implement something like a 'no data' message when a project wiki is missing and add a way for users with the required permission(s) to reinstate a new wiki by just letting them add new pages from the front-end while creating a new project wiki in the background automatically if one is not yet available. That would also take away the need for an additional notification message. What do you think?
Btw: I think you got it but to be certain, when you wrote ...deleting the wiki root page... -> it's actually about deleting the project's wiki itself. I've written about this is more detail in RE: Can't add deleted wiki Tab. (404).
Updated by Marius BĂLTEANU over 2 years ago
- File 34634.patch 34634.patch added
- Assignee deleted (
Mischa The Evil)
What do you think if we create a new empty Wiki after deletion?
Updated by Marius BĂLTEANU over 2 years ago
- Target version changed from 5.0.0 to 5.1.0
Updated by Mischa The Evil about 1 year ago
Marius BALTEANU wrote in #note-8:
What do you think if we create a new empty Wiki after deletion?
FWIW: I think that'll work too. Though, for clarity it might be better to change the name of the proposed class method Wiki.create_default_page to something more generic like just Wiki.create_default since we're actually creating a Wiki object, not an object associated with wiki pages (i.e. WikiPage).
Updated by Go MAEDA about 1 year ago
- Target version changed from 5.1.0 to 6.0.0
Updated by Marius BĂLTEANU about 1 year ago
Thanks for your feedback, Mischa. I will try to catch this in 5.1.0.
Updated by Marius BĂLTEANU about 1 year ago
- Status changed from Confirmed to Resolved
- Assignee set to Marius BĂLTEANU
- Target version changed from 6.0.0 to 5.1.0
- Resolution set to Fixed
I've committed the patch that reinitialise an empty wiki page after the project wiki is deleted. I don't see really necessary to have an extra message for this.
Updated by Mischa The Evil about 1 year ago
- Status changed from Resolved to Closed
Marius BALTEANU wrote in #note-13:
I've committed the patch that reinitialise an empty wiki page after the project wiki is deleted. I don't see really necessary to have an extra message for this.
Thanks! And I agree with you that an additional message would be unnecessary.
Updated by Mischa The Evil about 1 year ago
- Subject changed from Can't add deleted wiki Tab (404) to Deletion of project wiki leaves the project wiki inaccessible (404) until module reactivation
Change subject for changelog clarity.