Track parent page for internal links in rich text

Page chooser behaviour changes in Wagtail 1.5.1.
4n9le-bot · Jun 3, 2016 · 41c78ed · 41c78ed
1 parent 8a78c6b
commit 41c78ed
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/wagtailtinymce/static/wagtailtinymce/js/tinymce-plugins/wagtaillink.js b/wagtailtinymce/static/wagtailtinymce/js/tinymce-plugins/wagtaillink.js
@@ -37,6 +37,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
         a.setAttribute('href', pageData.url);
         if (pageData.id) {
             a.setAttribute('data-id', pageData.id);
+            a.setAttribute('data-parent-id', pageData.parentId);
             a.setAttribute('data-linktype', 'page');
             // If it's a link to an internal page, `pageData.title` will not use the link_text
             // like external and email responses do, overwriting selection text :(
@@ -71,11 +72,10 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
                 if ($targetNode.length) {
                     currentText = $targetNode.text();
                     var linkType = $targetNode.data('linktype');
-                    var pageId = $targetNode.data('id');
+                    var parentPageId = $targetNode.data('parent-id');
                     var href = $targetNode.attr('href');
-                    if (linkType == 'page' && pageId) {
-                        // TODO: Actually show the parent not the page itself.
-                        url = window.chooserUrls.pageChooser + pageId.toString() + '/';
+                    if (linkType == 'page' && parentPageId) {
+                        url = window.chooserUrls.pageChooser + parentPageId.toString() + '/';
                     }
                     else if (href.startsWith('mailto:')) {
                         url = window.chooserUrls.emailLinkChooser;