Skip to content

Commit

Permalink
Changed the 'override' word into 'overwrite' which makes more sense
Browse files Browse the repository at this point in the history
  • Loading branch information
AdrienCastex committed May 22, 2017
1 parent 7ff9fd9 commit f3caaab
Show file tree
Hide file tree
Showing 15 changed files with 17 additions and 17 deletions.
2 changes: 1 addition & 1 deletion lib/resource/IResource.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ export interface IResource {
fsManager: FSManager;
create(callback: SimpleCallback): any;
delete(callback: SimpleCallback): any;
moveTo(parent: IResource, newName: string, override: boolean, callback: SimpleCallback): any;
moveTo(parent: IResource, newName: string, overwrite: boolean, callback: SimpleCallback): any;
rename(newName: string, callback: Return2Callback<string, string>): any;
isSame(resource: IResource, callback: ReturnCallback<boolean>): any;
isOnTheSameFSWith(resource: IResource, callback: ReturnCallback<boolean>): any;
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/physical/PhysicalResource.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ export declare abstract class PhysicalResource extends StandardResource {
constructor(realPath: string, parent?: IResource, fsManager?: FSManager);
abstract create(callback: SimpleCallback): any;
abstract delete(callback: SimpleCallback): any;
moveTo(parent: IResource, newName: string, override: boolean, callback: SimpleCallback): void;
moveTo(parent: IResource, newName: string, overwrite: boolean, callback: SimpleCallback): void;
rename(newName: string, callback: Return2Callback<string, string>): void;
webName(callback: ReturnCallback<string>): void;
abstract type(callback: ReturnCallback<ResourceType>): any;
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/physical/PhysicalResource.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ var PhysicalResource = (function (_super) {
_this.name = path.basename(_this.realPath);
return _this;
}
PhysicalResource.prototype.moveTo = function (parent, newName, override, callback) {
PhysicalResource.prototype.moveTo = function (parent, newName, overwrite, callback) {
var _this = this;
if (parent === this.parent) {
this.rename(newName, function (e, oldName, newName) {
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/std/RootResource.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ export declare class RootResource extends StandardResource {
constructor();
create(callback: SimpleCallback): void;
delete(callback: SimpleCallback): void;
moveTo(parent: IResource, newName: string, override: boolean, callback: SimpleCallback): void;
moveTo(parent: IResource, newName: string, overwrite: boolean, callback: SimpleCallback): void;
rename(newName: string, callback: Return2Callback<string, string>): void;
webName(callback: ReturnCallback<string>): void;
type(callback: ReturnCallback<ResourceType>): void;
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/std/RootResource.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ var RootResource = (function (_super) {
RootResource.prototype.delete = function (callback) {
callback(Errors_1.Errors.InvalidOperation);
};
RootResource.prototype.moveTo = function (parent, newName, override, callback) {
RootResource.prototype.moveTo = function (parent, newName, overwrite, callback) {
callback(Errors_1.Errors.InvalidOperation);
};
RootResource.prototype.rename = function (newName, callback) {
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/std/StandardResource.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export declare abstract class StandardResource implements IResource {
getProperties(callback: ReturnCallback<object>): void;
abstract create(callback: SimpleCallback): any;
abstract delete(callback: SimpleCallback): any;
abstract moveTo(parent: IResource, newName: string, override: boolean, callback: SimpleCallback): any;
abstract moveTo(parent: IResource, newName: string, overwrite: boolean, callback: SimpleCallback): any;
abstract rename(newName: string, callback: Return2Callback<string, string>): any;
abstract append(data: Int8Array, targetSource: boolean, callback: SimpleCallback): any;
abstract write(data: Int8Array, targetSource: boolean, callback: SimpleCallback): any;
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/virtual/VirtualResource.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ export declare abstract class VirtualResource extends StandardResource {
constructor(name: string, parent?: IResource, fsManager?: FSManager);
create(callback: SimpleCallback): void;
delete(callback: SimpleCallback): void;
moveTo(parent: IResource, newName: string, override: boolean, callback: SimpleCallback): void;
moveTo(parent: IResource, newName: string, overwrite: boolean, callback: SimpleCallback): void;
rename(newName: string, callback: Return2Callback<string, string>): void;
webName(callback: ReturnCallback<string>): void;
abstract type(callback: ReturnCallback<ResourceType>): any;
Expand Down
2 changes: 1 addition & 1 deletion lib/resource/virtual/VirtualResource.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ var VirtualResource = (function (_super) {
VirtualResource.prototype.delete = function (callback) {
this.removeFromParent(callback);
};
VirtualResource.prototype.moveTo = function (parent, newName, override, callback) {
VirtualResource.prototype.moveTo = function (parent, newName, overwrite, callback) {
var _this = this;
if (parent === this.parent) {
this.rename(newName, function (e, oldName, newName) {
Expand Down
4 changes: 2 additions & 2 deletions lib/server/commands/Move.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ function default_1(arg, callback) {
return;
}
arg.requirePrivilege(['canMove'], r, function () {
var override = arg.findHeader('overwrite') === 'T';
var overwrite = arg.findHeader('overwrite') === 'T';
var destination = arg.findHeader('destination');
if (!destination) {
arg.setCode(WebDAVRequest_1.HTTPCodes.BadRequest);
Expand All @@ -22,7 +22,7 @@ function default_1(arg, callback) {
destination = new FSManager_1.FSPath(destination);
arg.server.getResourceFromPath(destination.getParent(), function (e, rDest) {
arg.requirePrivilege(['canAddChild'], rDest, function () {
r.moveTo(rDest, destination.fileName(), override, function (e) {
r.moveTo(rDest, destination.fileName(), overwrite, function (e) {
if (e)
arg.setCode(WebDAVRequest_1.HTTPCodes.InternalServerError);
else
Expand Down
2 changes: 1 addition & 1 deletion src/resource/IResource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export interface IResource
// ****************************** Actions ****************************** //
create(callback : SimpleCallback)
delete(callback : SimpleCallback)
moveTo(parent : IResource, newName : string, override : boolean, callback : SimpleCallback)
moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
rename(newName : string, callback : Return2Callback<string, string>)

// ****************************** Tests ****************************** //
Expand Down
2 changes: 1 addition & 1 deletion src/resource/physical/PhysicalResource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export abstract class PhysicalResource extends StandardResource
// ****************************** Actions ****************************** //
abstract create(callback : SimpleCallback)
abstract delete(callback : SimpleCallback)
moveTo(parent : IResource, newName : string, override : boolean, callback : SimpleCallback)
moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
{
if(parent === this.parent)
{
Expand Down
2 changes: 1 addition & 1 deletion src/resource/std/RootResource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export class RootResource extends StandardResource
{
callback(Errors.InvalidOperation)
}
moveTo(parent : IResource, newName : string, override : boolean, callback : SimpleCallback)
moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
{
callback(Errors.InvalidOperation)
}
Expand Down
2 changes: 1 addition & 1 deletion src/resource/std/StandardResource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ export abstract class StandardResource implements IResource
// ****************************** Actions ****************************** //
abstract create(callback : SimpleCallback)
abstract delete(callback : SimpleCallback)
abstract moveTo(parent : IResource, newName : string, override : boolean, callback : SimpleCallback)
abstract moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
abstract rename(newName : string, callback : Return2Callback<string, string>)

// ****************************** Content ****************************** //
Expand Down
2 changes: 1 addition & 1 deletion src/resource/virtual/VirtualResource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export abstract class VirtualResource extends StandardResource
{
this.removeFromParent(callback);
}
moveTo(parent : IResource, newName : string, override : boolean, callback : SimpleCallback)
moveTo(parent : IResource, newName : string, overwrite : boolean, callback : SimpleCallback)
{
if(parent === this.parent)
{
Expand Down
4 changes: 2 additions & 2 deletions src/server/commands/Move.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export default function(arg : MethodCallArgs, callback)
}

arg.requirePrivilege([ 'canMove' ], r, () => {
const override = arg.findHeader('overwrite') === 'T';
const overwrite = arg.findHeader('overwrite') === 'T';

let destination : any = arg.findHeader('destination');
if(!destination)
Expand All @@ -29,7 +29,7 @@ export default function(arg : MethodCallArgs, callback)

arg.server.getResourceFromPath(destination.getParent(), (e, rDest) => {
arg.requirePrivilege([ 'canAddChild' ], rDest, () => {
r.moveTo(rDest, destination.fileName(), override, (e) => {
r.moveTo(rDest, destination.fileName(), overwrite, (e) => {
if(e)
arg.setCode(HTTPCodes.InternalServerError)
else
Expand Down

0 comments on commit f3caaab

Please sign in to comment.