Merge pull request #815 from wmde/isForeign

Optimize EntityId::isForeign()

Author: mariushoch

RELEASE-NOTES.md

@@ -4,6 +4,7 @@
 
 * `TermList` now throws `InvalidArgumentException` when given non-iterable rather than failing silently
 * `SiteLinkList` now throws `InvalidArgumentException` when given non-iterable rather than failing silently
+* Slightly optimized `EntityId::isForeign()` by using `$this->repositoryName` instead of `$this->serialization`
 
 ## Version 9.1.0 (2019-01-24)
 

src/Entity/EntityId.php

@@ -164,15 +164,14 @@ public function getLocalPart() {
 	}
 
 	/**
-	 * Returns true iff EntityId::getRepoName returns a non-empty string.
+	 * Returns true iff EntityId::getRepositoryName returns a non-empty string.
 	 *
 	 * @since 6.2
 	 *
 	 * @return bool
 	 */
 	public function isForeign() {
-		// not actually using EntityId::getRepoName for performance reasons
-		return strpos( $this->serialization, ':' ) > 0;
+		return $this->repositoryName !== '';
 	}
 
 	/**