Feature #33820
closed
Auto complete wiki page links
Added by Mizuki ISHIKAWA almost 4 years ago. Updated about 3 years ago.
0%
Description
I think it would be useful to use the functionality of Tribute to auto complete the title of a wiki page at [[wiki page title]].
Files
| ScreenShot_2020-08-06_10.58.16.png (31.3 KB) ScreenShot_2020-08-06_10.58.16.png | Mizuki ISHIKAWA, 2020-08-06 04:02 | ||
| ScreenShot_2020-08-06_10.58.32.png (22 KB) ScreenShot_2020-08-06_10.58.32.png | Mizuki ISHIKAWA, 2020-08-06 04:02 | ||
| feature-33820.patch (15.9 KB) feature-33820.patch | Mizuki ISHIKAWA, 2020-08-06 04:04 | ||
| feature-33820-v2.diff (8.66 KB) feature-33820-v2.diff | Mizuki ISHIKAWA, 2020-11-18 01:13 | ||
| feature-33820-v3-with-test.patch (9.78 KB) feature-33820-v3-with-test.patch | Mizuki ISHIKAWA, 2021-02-15 01:23 | ||
| feature-33820-v4.patch (10.2 KB) feature-33820-v4.patch | Mizuki ISHIKAWA, 2021-02-25 01:40 |
Related issues
Updated by Mizuki ISHIKAWA almost 4 years ago
- File feature-33820.patch feature-33820.patch added
I attach a patch that adds this feature.
Updated by Go MAEDA almost 4 years ago
- Target version set to Candidate for next major release
Updated by Marius BĂLTEANU over 3 years ago
- Assignee set to Marius BĂLTEANU
Nice feature, let me find a better way to pass the auto complete endpoint for each case (issues, wiki pages, etc).
Updated by Marius BĂLTEANU over 3 years ago
- Related to Feature #33425: Auto complete wiki macros added
Updated by Marius BĂLTEANU over 3 years ago
- Related to Feature #31989: Inline issue auto complete (#) in fields with text-formatting enabled added
Updated by Marius BĂLTEANU over 3 years ago
Marius BALTEANU wrote:
Nice feature, let me find a better way to pass the auto complete endpoint for each case (issues, wiki pages, etc).
I've posted an alternative solution in #34122, please let me know what do you think about it.
Beside this point, the patch needs at least two changes:- Use a collection for triggers (see my note from #33425)
- Explore the possibility to show the wiki pages in parent - child relation or at least the parent if any in order to avoid confusion when there are multiple pages with the same title, but under different parent pages.
Updated by Go MAEDA over 3 years ago
Marius BALTEANU wrote:
Beside this point, the patch needs at least two changes:
- Use a collection for triggers (see my note from #33425)
- Explore the possibility to show the wiki pages in parent - child relation or at least the parent if any in order to avoid confusion when there are multiple pages with the same title, but under different parent pages.
I think the latter is not a problem. Wiki page names are unique in a project. You cannot create Wiki pages with the same name even if its parent pages are different.
Updated by Mizuki ISHIKAWA over 3 years ago
- File feature-33820-v2.diff feature-33820-v2.diff added
I fixed the patch by referring to the code of #34122.
Updated by Go MAEDA over 3 years ago
- Target version changed from Candidate for next major release to 4.2.0
Setting the target version to 4.2.0 again.
Updated by Marius BĂLTEANU over 3 years ago
Mizuki, can you add a system tests for this? you should have enough examples.
Updated by Mizuki ISHIKAWA about 3 years ago
Marius BALTEANU wrote:
Mizuki, can you add a system tests for this? you should have enough examples.
I have attached a patch with system test added.
There are already tests to check that tributes work on other pages, so I'm only added testing that the issues/new page can autocomplete the wiki page link.
Updated by Go MAEDA about 3 years ago
I found a security issue.
Auto complete for wiki pages should not suggest pages that the current user cannot view. The current patch always shows wiki pages even if the current user does not have "View wiki".
Updated by Mizuki ISHIKAWA about 3 years ago
- File feature-33820-v4.patch feature-33820-v4.patch added
Go MAEDA wrote:
I found a security issue.
Auto complete for wiki pages should not suggest pages that the current user cannot view. The current patch always shows wiki pages even if the current user does not have "View wiki".
Thank you for pointing that out.
I changed to return nothing when you do not have permission.
Updated by Go MAEDA about 3 years ago
- Subject changed from Auto Complete wiki page links to Auto complete wiki page links
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch. Thank you for improving Redmine.