Special:Browse use stuctured array instead of hash (#4452)

res/smw/special/ext.smw.special.browse.js

@@ -49,25 +49,14 @@
 		var self = this,
 			subject = self.context.data( 'subject' );
 
-		// Expect a serialization format (see DIWikiPage::getHash)
-		if ( subject.indexOf( "#" ) == -1 ) {
-			return this.context.find( '.smwb-status' )
-				.append(
-					mw.msg( 'smw-browse-api-subject-serialization-invalid' )
-				)
-				.addClass( 'smw-callout smw-callout-error' );
-		}
-
-		subject = subject.split( "#" );
-
 		self.api.post( {
 			action: "smwbrowse",
 			browse: "subject",
 			params: JSON.stringify( {
-				subject: subject[0],
-				ns: subject[1],
-				iw: subject[2],
-				subobject: subject[3],
+				subject: subject.dbkey,
+				ns: subject.ns,
+				iw: subject.iw,
+				subobject: subject.subobject,
 				options: self.options,
 				type: 'html'
 			} )

src/MediaWiki/Specials/Browse/HtmlBuilder.php

@@ -140,10 +140,18 @@ public function getOption( $key, $default = null ) {
 	 * @return string
 	 */
 	public function legacy() {
+
+		$subject = [
+			'dbkey' => $this->subject->getDBKey(),
+			'ns' => $this->subject->getNamespace(),
+			'iw' => $this->subject->getInterwiki(),
+			'subobject' => $this->subject->getSubobjectName(),
+		];
+
 		return Html::rawElement(
 			'div',
 			[
-				'data-subject' => $this->subject->getHash(),
+				'data-subject' => json_encode( $subject, JSON_UNESCAPED_UNICODE ),
 				'data-options' => json_encode( $this->options )
 			],
 			$this->buildHTML()
@@ -156,11 +164,19 @@ public function legacy() {
 	 * @return string
 	 */
 	public function placeholder() {
+
+		$subject = [
+			'dbkey' => $this->subject->getDBKey(),
+			'ns' => $this->subject->getNamespace(),
+			'iw' => $this->subject->getInterwiki(),
+			'subobject' => $this->subject->getSubobjectName(),
+		];
+
 		return Html::rawElement(
 			'div',
 			[
 				'class' => 'smwb-container',
-				'data-subject' => $this->subject->getHash(),
+				'data-subject' => json_encode( $subject, JSON_UNESCAPED_UNICODE ),
 				'data-options' => json_encode( $this->options )
 			],
 			Html::rawElement(

tests/phpunit/Unit/MediaWiki/Specials/SpecialBrowseTest.php

@@ -81,7 +81,7 @@ public function queryParameterProvider() {
 		$provider[] = [
 			'Foo/Bar',
 			[
-				'data-subject="Foo/Bar#0##"',
+				'data-subject="{"dbkey":"Foo\/Bar","ns":0,"iw":"","subobject":""}"',
 				'data-options="{"dir":null,"group":null,"printable":null,"offset":null,"including":null,"showInverse":false,"showAll":true,"showGroup":false,"showSort":false,"api":true,"valuelistlimit.out":"30","valuelistlimit.in":"20"}"'
 			]
 		];
@@ -90,7 +90,7 @@ public function queryParameterProvider() {
 		$provider[] = [
 			':Main-20Page-23_QUERY140d50d705e9566904fc4a877c755964',
 			[
-				'data-subject="Main_Page#0##_QUERY140d50d705e9566904fc4a877c755964"',
+				'data-subject="{"dbkey":"Main_Page","ns":0,"iw":"","subobject":"_QUERY140d50d705e9566904fc4a877c755964"}"',
 				'data-options="{"dir":null,"group":null,"printable":null,"offset":null,"including":null,"showInverse":false,"showAll":true,"showGroup":false,"showSort":false,"api":true,"valuelistlimit.out":"30","valuelistlimit.in":"20"}"'
 			]
 		];