Merge pull request #4300 from Snuffleupagus/issue-4296
Handle undefined url in LinkAnnotation
This commit is contained in:
		
						commit
						4c22afa380
					
				@ -644,7 +644,7 @@ var LinkAnnotation = (function LinkAnnotationClosure() {
 | 
				
			|||||||
        if (isName(url)) {
 | 
					        if (isName(url)) {
 | 
				
			||||||
          // Some bad PDFs do not put parentheses around relative URLs.
 | 
					          // Some bad PDFs do not put parentheses around relative URLs.
 | 
				
			||||||
          url = '/' + url.name;
 | 
					          url = '/' + url.name;
 | 
				
			||||||
        } else {
 | 
					        } else if (url) {
 | 
				
			||||||
          url = addDefaultProtocolToUrl(url);
 | 
					          url = addDefaultProtocolToUrl(url);
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
        // TODO: pdf spec mentions urls can be relative to a Base
 | 
					        // TODO: pdf spec mentions urls can be relative to a Base
 | 
				
			||||||
@ -684,7 +684,7 @@ var LinkAnnotation = (function LinkAnnotationClosure() {
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
  // Lets URLs beginning with 'www.' default to using the 'http://' protocol.
 | 
					  // Lets URLs beginning with 'www.' default to using the 'http://' protocol.
 | 
				
			||||||
  function addDefaultProtocolToUrl(url) {
 | 
					  function addDefaultProtocolToUrl(url) {
 | 
				
			||||||
    if (url.indexOf('www.') === 0) {
 | 
					    if (url && url.indexOf('www.') === 0) {
 | 
				
			||||||
      return ('http://' + url);
 | 
					      return ('http://' + url);
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
    return url;
 | 
					    return url;
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user