Skip to content

Commit d99d096

Browse files
Bugfix: ":jqmData(ajax='false')" might also match forms, which have neither of "src" or "href". In that case, thisUrl is undefined, and the following replace results in an error.
1 parent daffc98 commit d99d096

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

js/jquery.mobile.navigation.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,8 @@
495495
if( !$.support.dynamicBaseTag ) {
496496
var newPath = path.get( fileUrl );
497497
page.find( "[src], link[href], a[rel='external'], :jqmData(ajax='false'), a[target]" ).each(function() {
498-
var thisAttr = $( this ).is( '[href]' ) ? 'href' : 'src',
498+
var thisAttr = $( this ).is( '[href]' ) ? 'href' :
499+
$(this).is('[src]') ? 'src' : 'action',
499500
thisUrl = $( this ).attr( thisAttr );
500501

501502
// XXX_jblas: We need to fix this so that it removes the document

0 commit comments

Comments
 (0)