Skip to content

Commit

Permalink
[3.0][Config] Remove ResourceInterface::getResource() which was depre…
Browse files Browse the repository at this point in the history
…cated in 2.8
  • Loading branch information
mpdude authored and fabpot committed Sep 28, 2015
1 parent 8d7b498 commit 7cef180
Show file tree
Hide file tree
Showing 8 changed files with 5 additions and 26 deletions.
1 change: 1 addition & 0 deletions src/Symfony/Component/Config/CHANGELOG.md
Expand Up @@ -7,6 +7,7 @@ CHANGELOG
* removed `ReferenceDumper` class
* removed the `ResourceInterface::isFresh()` method
* removed `BCResourceInterfaceChecker` class
* removed `ResourceInterface::getResource()` method

2.8.0
-----
Expand Down
Expand Up @@ -42,7 +42,7 @@ public function __toString()
}

/**
* {@inheritdoc}
* @return string The file path to the resource
*/
public function getResource()
{
Expand Down
Expand Up @@ -45,7 +45,7 @@ public function __toString()
}

/**
* {@inheritdoc}
* @return string The file path to the resource
*/
public function getResource()
{
Expand Down
2 changes: 1 addition & 1 deletion src/Symfony/Component/Config/Resource/FileResource.php
Expand Up @@ -44,7 +44,7 @@ public function __toString()
}

/**
* {@inheritdoc}
* @return string|false The canonicalized, absolute path to the resource or false if the resource does not exist.
*/
public function getResource()
{
Expand Down
13 changes: 0 additions & 13 deletions src/Symfony/Component/Config/Resource/ResourceInterface.php
Expand Up @@ -30,17 +30,4 @@ interface ResourceInterface
* @return string A string representation unique to the underlying Resource
*/
public function __toString();

/**
* Returns the tied resource.
*
* @return mixed The resource
*
* @deprecated since 2.8, to be removed in 3.0. As there are many different kinds of resource,
* a single getResource() method does not make sense at the interface level. You
* can still call getResource() on implementing classes, probably after performing
* a type check. If you know the concrete type of Resource at hand, the return value
* of this method may make sense to you.
*/
public function getResource();
}
5 changes: 0 additions & 5 deletions src/Symfony/Component/Config/Tests/Resource/ResourceStub.php
Expand Up @@ -31,9 +31,4 @@ public function isFresh($timestamp)
{
return $this->fresh;
}

public function getResource()
{
return 'stub';
}
}
Expand Up @@ -50,7 +50,7 @@ public function __toString()
}

/**
* {@inheritdoc}
* @return array An array with two keys: 'prefix' for the prefix used and 'variables' containing all the variables watched by this resource
*/
public function getResource()
{
Expand Down
Expand Up @@ -288,10 +288,6 @@ public function isFresh($timestamp)
return false;
}

public function getResource()
{
}

public function __toString()
{
return '';
Expand Down

0 comments on commit 7cef180

Please sign in to comment.