Allow doc/section/files links to open in tabs

Closes #233 and might help #236

README.md

@@ -49,7 +49,7 @@ Heck, Documize will probably run just fine on a Raspberry Pi.
 
 ## Technology Stack
 
-- Go (v1.12.2)
+- Go (v1.12.3)
 - Ember JS (v3.8.0)
 
 ## Authentication Options

domain/attachment/endpoint.go

@@ -15,13 +15,14 @@ import (
 	"bytes"
 	"database/sql"
 	"fmt"
-	"github.com/documize/community/domain/auth"
-	"github.com/documize/community/model/space"
 	"io"
 	"mime"
 	"net/http"
 	"strings"
 
+	"github.com/documize/community/domain/auth"
+	"github.com/documize/community/model/space"
+
 	"github.com/documize/community/core/env"
 	"github.com/documize/community/core/request"
 	"github.com/documize/community/core/response"
@@ -161,7 +162,7 @@ func (h *Handler) Download(w http.ResponseWriter, r *http.Request) {
 		canDownload = true
 	}
 
-	if  len(secureToken) == 0 && len(authToken) == 0 {
+	if !canDownload && len(secureToken) == 0 && len(authToken) == 0 {
 		h.Runtime.Log.Error("get attachment received no access token", err)
 		response.WriteForbiddenError(w)
 		return

domain/link/endpoint.go

@@ -13,9 +13,12 @@ package link
 
 import (
 	"database/sql"
+	"fmt"
 	"net/http"
 	"net/url"
 
+	"github.com/documize/community/core/stringutil"
+
 	"github.com/documize/community/core/env"
 	"github.com/documize/community/core/request"
 	"github.com/documize/community/core/response"
@@ -160,3 +163,55 @@ func (h *Handler) SearchLinkCandidates(w http.ResponseWriter, r *http.Request) {
 
 	response.WriteJSON(w, payload)
 }
+
+// GetLink returns link object for given ID.
+func (h *Handler) GetLink(w http.ResponseWriter, r *http.Request) {
+	method := "link.GetLink"
+	ctx := domain.GetRequestContext(r)
+
+	// Param check.
+	linkID := request.Param(r, "linkID")
+	if len(linkID) == 0 {
+		response.WriteMissingDataError(w, method, "linkID")
+		return
+	}
+
+	// Load link record.
+	link, err := h.Store.Link.GetLink(ctx, linkID)
+	if err != nil {
+		response.WriteServerError(w, method, err)
+		return
+	}
+
+	// Check document permissions.
+	if !permission.CanViewDocument(ctx, *h.Store, link.SourceDocumentID) {
+		response.WriteForbiddenError(w)
+		return
+	}
+
+	// Build URL for link
+	url := ""
+
+	// Jump-to-document link type.
+	if link.LinkType == "document" {
+		doc, err := h.Store.Document.Get(ctx, link.TargetDocumentID)
+		if err != nil {
+			response.WriteString(w, url)
+		}
+		url = ctx.GetAppURL(fmt.Sprintf("s/%s/%s/d/%s/%s",
+			doc.SpaceID, doc.SpaceID, doc.RefID, stringutil.MakeSlug(doc.Name)))
+	}
+
+	// Jump-to-section link type.
+	if link.LinkType == "section" || link.LinkType == "tab" {
+		doc, err := h.Store.Document.Get(ctx, link.TargetDocumentID)
+		if err != nil {
+			response.WriteString(w, url)
+		}
+		url = ctx.GetAppURL(fmt.Sprintf("s/%s/%s/d/%s/%s?currentPageId=%s",
+			doc.SpaceID, doc.SpaceID, doc.RefID,
+			stringutil.MakeSlug(doc.Name), link.TargetID))
+	}
+
+	response.WriteString(w, url)
+}

domain/link/store.go

@@ -46,6 +46,25 @@ func (s Store) Add(ctx domain.RequestContext, l link.Link) (err error) {
 	return
 }
 
+// GetLink returns specified link.
+func (s Store) GetLink(ctx domain.RequestContext, linkID string) (l link.Link, err error) {
+	err = s.Runtime.Db.Get(&l, s.Bind(`
+		select c_refid AS refid, c_orgid AS orgid, c_spaceid AS spaceid, c_userid AS userid,
+        c_sourcedocid AS sourcedocumentid, c_sourcesectionid AS sourcesectionid,
+        c_targetdocid AS targetdocumentid, c_targetid AS targetid, c_externalid AS externalid,
+        c_type as linktype, c_orphan As orphan, c_created AS created, c_revised AS revised
+		FROM dmz_doc_link
+		WHERE c_orgid=? AND c_refid=?`),
+		ctx.OrgID, linkID)
+
+	if err != nil {
+		err = errors.Wrapf(err, "select link %s", linkID)
+		return
+	}
+
+	return
+}
+
 // GetDocumentOutboundLinks returns outbound links for specified document.
 func (s Store) GetDocumentOutboundLinks(ctx domain.RequestContext, documentID string) (links []link.Link, err error) {
 	err = s.Runtime.Db.Select(&links, s.Bind(`

domain/store/storer.go

@@ -215,6 +215,7 @@ type AttachmentStorer interface {
 type LinkStorer interface {
 	Add(ctx domain.RequestContext, l link.Link) (err error)
 	SearchCandidates(ctx domain.RequestContext, keywords string) (docs []link.Candidate, pages []link.Candidate, attachments []link.Candidate, err error)
+	GetLink(ctx domain.RequestContext, linkID string) (l link.Link, err error)
 	GetDocumentOutboundLinks(ctx domain.RequestContext, documentID string) (links []link.Link, err error)
 	GetPageLinks(ctx domain.RequestContext, documentID, pageID string) (links []link.Link, err error)
 	MarkOrphanDocumentLink(ctx domain.RequestContext, documentID string) (err error)

gui/app/pods/jumpto/controller.js

@@ -0,0 +1,15 @@
+// Copyright 2016 Documize Inc. <legal@documize.com>. All rights reserved.
+//
+// This software (Documize Community Edition) is licensed under
+// GNU AGPL v3 http://www.gnu.org/licenses/agpl-3.0.en.html
+//
+// You can operate outside the AGPL restrictions by purchasing
+// Documize Enterprise Edition and obtaining a commercial license
+// by contacting <sales@documize.com>.
+//
+// https://documize.com
+
+import Controller from '@ember/controller';
+
+export default Controller.extend({
+});

gui/app/pods/jumpto/route.js

@@ -0,0 +1,37 @@
+// Copyright 2016 Documize Inc. <legal@documize.com>. All rights reserved.
+//
+// This software (Documize Community Edition) is licensed under
+// GNU AGPL v3 http://www.gnu.org/licenses/agpl-3.0.en.html
+//
+// You can operate outside the AGPL restrictions by purchasing
+// Documize Enterprise Edition and obtaining a commercial license
+// by contacting <sales@documize.com>.
+//
+// https://documize.com
+
+import { inject as service } from '@ember/service';
+import { Promise } from 'rsvp';
+import AuthenticatedRouteMixin from 'ember-simple-auth/mixins/authenticated-route-mixin';
+import Route from '@ember/routing/route';
+
+export default Route.extend(AuthenticatedRouteMixin, {
+	linkSvc: service('link'),
+	jumpToLink: '',
+
+	beforeModel: function () {
+		// let jumpType = this.paramsFor('jumpto').jump_type;
+		let jumpId = this.paramsFor('jumpto').jump_id;
+
+		return new Promise((resolve) => {
+			this.get('linkSvc').fetchLinkUrl(jumpId).then((link) => {
+				this.set('jumpToLink', link);
+				resolve();
+			});
+		});
+	},
+
+	model: function () {
+		let link = this.get('jumpToLink');
+		window.location.href = link;
+	}
+});

gui/app/router.js

@@ -129,6 +129,10 @@ export default Router.map(function () {
 		path: 'secure/:token'
 	});
 
+	this.route('jumpto', {
+		path: 'link/:jump_type/:jump_id'
+	});
+
 	this.route(
 		'auth',
 		{

gui/app/services/link.js

@@ -35,6 +35,16 @@ export default Service.extend(Notifier, {
 		});
 	},
 
+	// Returns link URL for specified link.
+	fetchLinkUrl(linkId) {
+		return this.get('ajax').request(`link/${linkId}`, {
+			method: 'GET',
+			dataType: 'text'
+		}).then((response) => {
+			return response;
+		});
+	},	
+
 	// Returns keyword-based candidates
 	searchCandidates(keywords) {
 		let url = "links?keywords=" + encodeURIComponent(keywords);

server/routing/routes.go

@@ -187,6 +187,7 @@ func RegisterEndpoints(rt *env.Runtime, s *store.Store) {
 
 	AddPrivate(rt, "links/{spaceID}/{documentID}/{pageID}", []string{"GET", "OPTIONS"}, nil, link.GetLinkCandidates)
 	AddPrivate(rt, "links", []string{"GET", "OPTIONS"}, nil, link.SearchLinkCandidates)
+	AddPrivate(rt, "link/{linkID}", []string{"GET", "OPTIONS"}, nil, link.GetLink)
 	AddPrivate(rt, "documents/{documentID}/links", []string{"GET", "OPTIONS"}, nil, document.DocumentLinks)
 
 	AddPrivate(rt, "pin/{userID}", []string{"POST", "OPTIONS"}, nil, pin.Add)