Add support to know where the storage test comes from

This commit is contained in:
Juan Pablo Villafáñez 2016-06-07 16:53:16 +02:00
parent f28f6ad2df
commit 8526bbb347
7 changed files with 24 additions and 16 deletions

View File

@ -299,7 +299,8 @@ StorageConfig.prototype = {
mountPoint: this.mountPoint,
backend: this.backend,
authMechanism: this.authMechanism,
backendOptions: this.backendOptions
backendOptions: this.backendOptions,
origin: 'settings'
};
if (this.id) {
data.id = this.id;
@ -326,6 +327,7 @@ StorageConfig.prototype = {
$.ajax({
type: 'GET',
url: OC.generateUrl(this._url + '/{id}', {id: this.id}),
data: {'origin': 'settings'},
success: options.success,
error: options.error
});
@ -908,6 +910,7 @@ MountConfigListView.prototype = _.extend({
$.ajax({
type: 'GET',
url: OC.generateUrl('apps/files_external/userglobalstorages'),
data: {'origin' : 'settings'},
contentType: 'application/json',
success: function(result) {
var onCompletion = jQuery.Deferred();

View File

@ -78,6 +78,7 @@ OCA.External.StatusManager = {
defObj = $.ajax({
type: 'GET',
url: OC.webroot + '/index.php/apps/files_external/' + ((mountData.type === 'personal') ? 'userstorages' : 'userglobalstorages') + '/' + mountData.id,
data: {'origin' : 'statusmanager'},
success: function (response) {
if (response && response.status === 0) {
self.mountStatus[mountData.mount_point] = response;

View File

@ -139,7 +139,8 @@ class GlobalStoragesController extends StoragesController {
$mountOptions,
$applicableUsers,
$applicableGroups,
$priority
$priority,
$origin = null
) {
$storage = $this->createStorage(
$mountPoint,
@ -172,7 +173,7 @@ class GlobalStoragesController extends StoragesController {
);
}
$this->updateStorageStatus($storage);
$this->updateStorageStatus($storage, $origin);
return new DataResponse(
$storage,

View File

@ -238,7 +238,7 @@ abstract class StoragesController extends Controller {
*
* @param StorageConfig $storage storage configuration
*/
protected function updateStorageStatus(StorageConfig &$storage) {
protected function updateStorageStatus(StorageConfig &$storage, $origin = null) {
try {
$this->manipulateStorageConfig($storage);
@ -249,7 +249,8 @@ abstract class StoragesController extends Controller {
\OC_Mount_Config::getBackendStatus(
$backend->getStorageClass(),
$storage->getBackendOptions(),
false
false,
$origin
)
);
} catch (InsufficientDataForMeaningfulAnswerException $e) {
@ -293,11 +294,11 @@ abstract class StoragesController extends Controller {
*
* @return DataResponse
*/
public function show($id) {
public function show($id, $origin = null) {
try {
$storage = $this->service->getStorage($id);
$this->updateStorageStatus($storage);
$this->updateStorageStatus($storage, $origin);
} catch (NotFoundException $e) {
return new DataResponse(
[

View File

@ -111,11 +111,11 @@ class UserGlobalStoragesController extends StoragesController {
*
* @NoAdminRequired
*/
public function show($id) {
public function show($id, $origin = null) {
try {
$storage = $this->service->getStorage($id);
$this->updateStorageStatus($storage);
$this->updateStorageStatus($storage, $origin);
} catch (NotFoundException $e) {
return new DataResponse(
[
@ -146,7 +146,8 @@ class UserGlobalStoragesController extends StoragesController {
*/
public function update(
$id,
$backendOptions
$backendOptions,
$origin = null
) {
try {
$storage = $this->service->getStorage($id);

View File

@ -101,8 +101,8 @@ class UserStoragesController extends StoragesController {
*
* {@inheritdoc}
*/
public function show($id) {
return parent::show($id);
public function show($id, $origin = null) {
return parent::show($id, $origin);
}
/**
@ -170,7 +170,8 @@ class UserStoragesController extends StoragesController {
$backend,
$authMechanism,
$backendOptions,
$mountOptions
$mountOptions,
$origin = null
) {
$storage = $this->createStorage(
$mountPoint,
@ -200,7 +201,7 @@ class UserStoragesController extends StoragesController {
);
}
$this->updateStorageStatus($storage);
$this->updateStorageStatus($storage, $origin);
return new DataResponse(
$storage,

View File

@ -215,7 +215,7 @@ class OC_Mount_Config {
* @return int see self::STATUS_*
* @throws Exception
*/
public static function getBackendStatus($class, $options, $isPersonal) {
public static function getBackendStatus($class, $options, $isPersonal, $origin = null) {
if (self::$skipTest) {
return StorageNotAvailableException::STATUS_SUCCESS;
}
@ -228,7 +228,7 @@ class OC_Mount_Config {
$storage = new $class($options);
try {
$result = $storage->test($isPersonal);
$result = $storage->test($isPersonal, $origin);
$storage->setAvailability($result);
if ($result) {
return StorageNotAvailableException::STATUS_SUCCESS;