Merge pull request #15 from wmde/removephp

Removed ParseValue API module

Author: addshore

DataValuesJavascript.php

@@ -34,14 +34,6 @@
 	include( __DIR__ . '/src/valueFormatters/resources.mw.php' )
 );
 
-// API module registration
-$GLOBALS['wgAPIModules']['parsevalue'] = 'ValueParsers\ApiParseValue';
-
-/**
- * @deprecated since 0.1 This is a global registry that provides no control over object lifecycle
- */
-$GLOBALS['wgValueParsers'] = array();
-
 /**
  * Register QUnit test base classes used by test modules in dependent components.
  * @see https://www.mediawiki.org/wiki/Manual:Hooks/ResourceLoaderTestModules

README.md

@@ -11,14 +11,8 @@ On [Packagist](https://packagist.org/packages/data-values/javascript):
 
 ## Technical debt
 
-TODO: remove dependency on ApiParserValue from the JS so ApiParseValue
-can be removed. ApiParseValue is the only PHP class in this package and
-is the only thing that depends on the DataValues PHP components.
-
 TODO: write high level description and documentation in this README file.
 
-TODO: make tests runnable without needing MW and a browser
-
 TODO: have tests run on CI
 
 ## Release notes

composer.json

@@ -26,15 +26,11 @@
 		"irc": "irc://irc.freenode.net/wikidata"
 	},
 	"require": {
-		"composer/installers": ">=1.0.1",
-		"data-values/interfaces": "~0.1"
+		"composer/installers": ">=1.0.1"
 	},
 	"autoload": {
 		"files" : [
 			"DataValuesJavascript.php"
-		],
-		"classmap": [
-			"php/src/"
 		]
 	}
 }

php/src/ApiParseValue.php

@@ -5,8 +5,8 @@
 }
 
 $pwd = exec( 'pwd' );
-chdir( __DIR__ . '/../..' );
+chdir( __DIR__ );
 passthru( 'composer update' );
 chdir( $pwd );
 
-require_once( __DIR__ . '/../../vendor/autoload.php' );
+require_once( __DIR__ . '/vendor/autoload.php' );