From 8cb84eac6821f6c14f68936df1d7a6765ae96af5 Mon Sep 17 00:00:00 2001
From: Vojta Jina
Date: Wed, 18 May 2011 11:13:46 +0200
Subject: Log warning for every non existing link instead of throwing exception

Guess we don't want to stop doc generating process because of non-existing link, so just log warning and continue...
---
 docs/src/ngdoc.js | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

(limited to 'docs/src')

diff --git a/docs/src/ngdoc.js b/docs/src/ngdoc.js
index c2f5256a..c814c968 100644
--- a/docs/src/ngdoc.js
+++ b/docs/src/ngdoc.js
@@ -126,12 +126,16 @@ Doc.prototype = {
         text = text.replace(/{@link\s+([^\s}]+)\s*([^}]*?)\s*}/g,
           function(_all, url, title){
             var isFullUrl = url.match(IS_URL),
+                // FIXME(vojta) angular link could be api.angular now with sections
                 isAngular = url.match(IS_ANGULAR);
 
-            url = isFullUrl ? url : self.sectionHuristic(url);
-            self.links.push(url);
+            if (!isFullUrl) {
+              // TODO(vojta) there could be relative link, but not angular
+              // do we want to store all links (and check even the full links like http://github.com ?
+              self.links.push(self.sectionHuristic(url));
+            }
 
-            return '<a href="' + (isFullUrl ? '' + url : '#!' + url) + '">'
+            return '<a href="' + (isFullUrl ? '' + url : '#!' + self.sectionHuristic(url)) + '">'
               + (isAngular ? '<code>' : '')
               + (title || url).replace(/\n/g, ' ')
               + (isAngular ? '</code>' : '')
@@ -680,11 +684,11 @@ function merge(docs){
       i++;
     }
   }
-  
+
   // check links
   docs.forEach(function(doc) {
     doc.links.forEach(function(link) {
-      if (!byFullId[link]) throw 'Not existing link "' + link + '" in ' + doc.section + '/' + doc.id;
+      if (!byFullId[link]) console.log('WARNING: Non existing link "' + link + '" in ' + doc.section + '/' + doc.id);
     });
   });
 
-- 
cgit v1.2.3