Skip to content

Commit d5474a4

Browse files
authored
Remove deprecated Item::getSiteLinks (#739)
This fixes #733.
1 parent e948220 commit d5474a4

2 files changed

Lines changed: 0 additions & 25 deletions

File tree

src/Entity/Item.php

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -231,16 +231,6 @@ public function removeSiteLink( $siteId ) {
231231
$this->siteLinks->removeLinkWithSiteId( $siteId );
232232
}
233233

234-
/**
235-
* @deprecated since 0.8, use getSiteLinkList() instead,
236-
* @since 0.6
237-
*
238-
* @return SiteLink[]
239-
*/
240-
public function getSiteLinks() {
241-
return array_values( iterator_to_array( $this->siteLinks ) );
242-
}
243-
244234
/**
245235
* @deprecated since 0.8, use getSiteLinkList()->getBySiteId() instead.
246236
* @since 0.6

tests/unit/Entity/ItemTest.php

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -117,21 +117,6 @@ public function simpleSiteLinkProvider() {
117117
return $argLists;
118118
}
119119

120-
/**
121-
* @dataProvider simpleSiteLinksProvider
122-
*/
123-
public function testGetSiteLinks() {
124-
$siteLinks = func_get_args();
125-
$item = new Item();
126-
127-
foreach ( $siteLinks as $siteLink ) {
128-
$item->getSiteLinkList()->addSiteLink( $siteLink );
129-
}
130-
131-
$this->assertInternalType( 'array', $item->getSiteLinks() );
132-
$this->assertEquals( $siteLinks, $item->getSiteLinks() );
133-
}
134-
135120
public function simpleSiteLinksProvider() {
136121
$argLists = [];
137122

0 commit comments

Comments
 (0)