Skip to content

Deprecated some methods of OCP\Files #8922

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rullzer merged 1 commit into from
Mar 22, 2018
+15 −0
Merged

Deprecated some methods of OCP\Files #8922

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions lib/private/App/CodeChecker/DeprecationCheck.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ protected function getLocalClasses() {
'OCP\User' => '13.0.0',
'OCP\BackgroundJob' => '14.0.0',
'OCP\App' => '14.0.0',
'OCP\Files' => '14.0.0',
];
}

Expand Down Expand Up @@ -174,6 +175,13 @@ protected function getLocalMethods() {
'OCP\Util::mb_substr_replace' => '8.2.0',
'OCP\Util::sendMail' => '8.1.0',
'OCP\Util::writeLog' => '13.0.0',

'OCP\Files::rmdirr' => '14.0.0',
'OCP\Files::getMimeType' => '14.0.0',
'OCP\Files::searchByMime' => '14.0.0',
'OCP\Files::streamCopy' => '14.0.0',
'OCP\Files::buildNotExistingFileName' => '14.0.0',
'OCP\Files::getStorage' => '14.0.0',
];
}
}
7 changes: 7 additions & 0 deletions lib/public/Files.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,12 +44,14 @@
* This class provides access to the internal filesystem abstraction layer. Use
* this class exlusively if you want to access files
* @since 5.0.0
* @deprecated 14.0.0
*/
class Files {
/**
* Recusive deletion of folders
* @return bool
* @since 5.0.0
* @deprecated 14.0.0
*/
static public function rmdirr( $dir ) {
return \OC_Helper::rmdirr( $dir );
Expand All @@ -61,6 +63,7 @@ static public function rmdirr( $dir ) {
* @return string
* does NOT work for ownClouds filesystem, use OC_FileSystem::getMimeType instead
* @since 5.0.0
* @deprecated 14.0.0
*/
static public function getMimeType( $path ) {
return \OC::$server->getMimeTypeDetector()->detect($path);
Expand All @@ -71,6 +74,7 @@ static public function getMimeType( $path ) {
* @param string $mimetype
* @return array
* @since 6.0.0
* @deprecated 14.0.0
*/
static public function searchByMime($mimetype) {
return \OC\Files\Filesystem::searchByMime($mimetype);
Expand All @@ -82,6 +86,7 @@ static public function searchByMime($mimetype) {
* @param resource $target
* @return int the number of bytes copied
* @since 5.0.0
* @deprecated 14.0.0
*/
public static function streamCopy( $source, $target ) {
list($count, ) = \OC_Helper::streamCopy( $source, $target );
Expand All @@ -94,6 +99,7 @@ public static function streamCopy( $source, $target ) {
* @param string $filename
* @return string
* @since 5.0.0
* @deprecated 14.0.0 use getNonExistingName of the OCP\Files\Folder object
*/
public static function buildNotExistingFileName($path, $filename) {
return \OC_Helper::buildNotExistingFileName($path, $filename);
Expand All @@ -105,6 +111,7 @@ public static function buildNotExistingFileName($path, $filename) {
* @param string $app
* @return \OC\Files\View
* @since 5.0.0
* @deprecated 14.0.0 use IAppData instead
*/
public static function getStorage($app) {
return \OC_App::getStorage( $app );
Expand Down