Various small fixes. Bumping to 1.2.0 to indicate minor backwards incompatibility.

Author: Alexia

.DS_Store

@@ -1,4 +1,5 @@
+.svn
+*~
+*.kate-swp
+.*.swp
 .DS_Store
-.idea
-*.sublime-workspace
-*.sublime-project

.gitignore

@@ -181,7 +181,7 @@ static public function generateSpriteOutput(Parser &$parser, PPFrame $frame, $ar
 					return self::makeErrorBox();
 				}
 
-				$html = $spriteSheet->getSpriteHtmlFromName($spriteName->getName(), $parameters['resize']['number']);
+				$html = $spriteSheet->getSpriteHtmlFromName($spriteName->getName(), isset($parameters['resize']['number']) ? $parameters['resize']['number'] : null);
 			} else {
 				if (!isset($parameters['column']['integer']) || $parameters['column']['integer'] < 0) {
 					self::setError('spritesheet_error_invalid_option', ['column', $parameters['column']['integer']]);
@@ -266,7 +266,7 @@ static public function generateSliceOutput(Parser &$parser, PPFrame $frame, $arg
 						return self::makeErrorBox();
 					}
 
-					$html = $spriteSheet->getSliceHtmlFromName($sliceName->getName(), $parameters['resize']['number'], $pixelMode);
+					$html = $spriteSheet->getSliceHtmlFromName($sliceName->getName(), isset($parameters['resize']['number']) ? $parameters['resize']['number'] : null, $pixelMode);
 				} else {
 					//The unit of measure is allowed to be specified, but they must match to be valid.
 					$unitParams = ['x', 'y', 'width', 'height'];
@@ -367,7 +367,7 @@ static private function cleanAndSetupParameters($arguments, PPFrame $frame) {
 			if (isset($validParameters[$parameter])) {
 				$cleanParameterOptions[$parameter] = $option;
 
-				if (is_array($validParameters[$parameter]['values'])) {
+				if (isset($validParameters[$parameter]['values']) && is_array($validParameters[$parameter]['values'])) {
 					if (!in_array($option, $validParameters[$parameter]['values'])) {
 						//Throw an error.
 						unset($cleanParameterOptions[$parameter]);
@@ -574,7 +574,7 @@ static public function onPageRenderingHash(&$hash, User $user, &$forOptions) {
 	 * @param	string	Reason given by the user performing the move.
 	 * @return	boolean True
 	 */
-	static public function onTitleMoveComplete(Title &$oldTitle, Title &$newTitle, User &$user, $oldId, $newId, $reason = null) {
+	static public function onTitleMoveComplete(Title &$oldTitle, Title &$newTitle, User $user, $oldId, $newId, $reason = null) {
 		$spriteSheet = SpriteSheet::newFromTitle($oldTitle);
 
 		if (!$spriteSheet || !$spriteSheet->exists()) {

SpriteSheet.hooks.php

@@ -313,7 +313,7 @@ public function setId($id) {
 	 * @return	integer	Sprite Sheet ID
 	 */
 	public function getId() {
-		return intval($this->data['spritesheet_id']);
+		return !isset($this->data['spritesheet_id']) ? 0 : intval($this->data['spritesheet_id']);
 	}
 
 	/**
@@ -367,7 +367,7 @@ public function setColumns($columns) {
 	 * @return	integer	Columns
 	 */
 	public function getColumns() {
-		return intval($this->data['columns']);
+		return !isset($this->data['columns']) ? 0 : intval($this->data['columns']);
 	}
 
 	/**
@@ -388,7 +388,7 @@ public function setRows($rows) {
 	 * @return	integer	Rows
 	 */
 	public function getRows() {
-		return intval($this->data['rows']);
+		return !isset($this->data['rows']) ? 0 : intval($this->data['rows']);
 	}
 
 	/**
@@ -409,7 +409,7 @@ public function setInset($inset) {
 	 * @return	integer	Inset
 	 */
 	public function getInset() {
-		return intval($this->data['inset']);
+		return !isset($this->data['inset']) ? 0 : intval($this->data['inset']);
 	}
 
 	/**

SpriteSheet.i18n.php

@@ -1,6 +1,6 @@
 {
 	"name": "SpriteSheet",
-	"version": "1.1.5",
+	"version": "1.2.0",
 	"author": [
 		"Curse Inc. Wiki Platform Team",
 		"Alexia E. Smith"
@@ -80,7 +80,11 @@
 				"click_grid_for_preview",
 				"no_results_named_sprites"
 			],
-			"position": "top"
+			"position": "top",
+			"targets": [
+				"desktop",
+				"mobile"
+			]
 		}
 	},
 	"ResourceFileModulePaths": {
@@ -107,5 +111,5 @@
 			"SpriteSheetHooks::onLoadExtensionSchemaUpdates"
 		]
 	},
-	"manifest_version": 1
-}
+	"manifest_version": 2
+}