make sure unlinking and relinking properly invalidates deps

src/model/LinkModel.js

@@ -154,7 +154,10 @@ export default class LinkModel extends ModelBase {
 
 		if ( this.rootLink ) this.addShuffleTask( () => {
 			this.relinked();
-			if ( !safe ) this.notifyUpstream();
+			if ( !safe ) {
+				this.markedAll();
+				this.notifyUpstream();
+			}
 		});
 	}
 
@@ -193,7 +196,7 @@ ModelBase.prototype.link = function link ( model, keypath, options ) {
 	lnk.implicit = options && options.implicit;
 	lnk.sourcePath = keypath;
 	lnk.rootLink = true;
-	if ( this._link ) this._link.relinking( model, true, false );
+	if ( this._link ) this._link.relinking( model, false );
 	this.rebind( lnk, this, false );
 	fireShuffleTasks();
 
@@ -208,7 +211,7 @@ ModelBase.prototype.unlink = function unlink () {
 	if ( this._link ) {
 		const ln = this._link;
 		this._link = undefined;
-		ln.rebind( this, this._link );
+		ln.rebind( this, ln, false );
 		fireShuffleTasks();
 		ln.teardown();
 		this.notifyUpstream();

tests/browser/methods/link.js

@@ -160,4 +160,31 @@ export default function() {
 			r2.link( 'foo.bar', 'foo.bar', { ractive: r1 });
 		}, /keypath cannot be linked to itself/i);
 	});
+
+	test( `unlinking and relinking linked links properly updates deps`, t => {
+		const r1 = new Ractive({
+			data: { foo: { bar: 'a' }, bat: { bar: 'b' } }
+		});
+		const r2 = new Ractive({
+			target: fixture,
+			template: '{{~/link.bar}}'
+		});
+
+		r2.link( 'baz', 'link', { ractive: r1 } );
+		r1.link( 'foo', 'baz' );
+
+		t.htmlEqual( fixture.innerHTML, 'a' );
+
+		r1.unlink( 'baz' );
+		t.equal( fixture.innerHTML, '' );
+
+		r1.link( 'bat', 'baz' );
+		t.htmlEqual( fixture.innerHTML, 'b' );
+
+		r1.unlink( 'baz' );
+		t.equal( fixture.innerHTML, '' );
+
+		r1.link( 'bat', 'baz' );
+		t.htmlEqual( fixture.innerHTML, 'b' );
+	});
 }