[ADF-1458] wrongly named input attribute - renamed disabled into disable (#2270)

* [ADF-1458] wrongly named input attribute - renamed disabled into disable

* [ADF 1458] renamed disabled attribute in disable
This commit is contained in:
Vito
2017-09-01 13:39:09 -07:00
committed by Mario Romano
parent 893252dd7a
commit 1f66cf8c56
6 changed files with 24 additions and 24 deletions

View File

@@ -268,10 +268,10 @@
</md-input-container> </md-input-container>
</section> </section>
<div *ngIf="!acceptedFilesTypeShow"> <div *ngIf="!acceptedFilesTypeShow">
<alfresco-upload-button <adf-upload-button
#uploadButton #uploadButton
tooltip="Custom tooltip" tooltip="Custom tooltip"
[disabled]="!enableUpload" [disable]="!enableUpload"
data-automation-id="multiple-file-upload" data-automation-id="multiple-file-upload"
[rootFolderId]="documentList.currentFolderId" [rootFolderId]="documentList.currentFolderId"
[multipleFiles]="multipleFileUpload" [multipleFiles]="multipleFileUpload"
@@ -280,13 +280,13 @@
[adf-node-permission]="'create'" [adf-node-permission]="'create'"
[adf-nodes]="getCurrentDocumentListNode()" [adf-nodes]="getCurrentDocumentListNode()"
(permissionEvent)="handlePermissionError($event)"> (permissionEvent)="handlePermissionError($event)">
</alfresco-upload-button> </adf-upload-button>
</div> </div>
<div *ngIf="acceptedFilesTypeShow"> <div *ngIf="acceptedFilesTypeShow">
<alfresco-upload-button <alfresco-upload-button
#uploadButton #uploadButton
tooltip="Custom tooltip" tooltip="Custom tooltip"
[disabled]="!enableUpload" [disable]="!enableUpload"
data-automation-id="multiple-file-upload" data-automation-id="multiple-file-upload"
[rootFolderId]="documentList.currentFolderId" [rootFolderId]="documentList.currentFolderId"
[acceptedFilesType]="acceptedFilesType" [acceptedFilesType]="acceptedFilesType"

View File

@@ -23,7 +23,7 @@ import { NodePermissionDirective, NodePermissionSubject } from './node-permissio
selector: 'adf-text-subject' selector: 'adf-text-subject'
}) })
class TestComponent implements NodePermissionSubject { class TestComponent implements NodePermissionSubject {
disabled: boolean = false; disable: boolean = false;
} }
describe('NodePermissionDirective', () => { describe('NodePermissionDirective', () => {
@@ -126,13 +126,13 @@ describe('NodePermissionDirective', () => {
spyOn(contentService, 'hasPermission').and.returnValue(false); spyOn(contentService, 'hasPermission').and.returnValue(false);
let testComponent = new TestComponent(); let testComponent = new TestComponent();
testComponent.disabled = false; testComponent.disable = false;
const directive = new NodePermissionDirective(null, null, contentService, testComponent); const directive = new NodePermissionDirective(null, null, contentService, testComponent);
directive.nodes = <any> [{}, {}]; directive.nodes = <any> [{}, {}];
directive.updateElement(); directive.updateElement();
expect(testComponent.disabled).toBeTruthy(); expect(testComponent.disable).toBeTruthy();
}); });
it('enables decorated component', () => { it('enables decorated component', () => {
@@ -140,13 +140,13 @@ describe('NodePermissionDirective', () => {
spyOn(contentService, 'hasPermission').and.returnValue(true); spyOn(contentService, 'hasPermission').and.returnValue(true);
let testComponent = new TestComponent(); let testComponent = new TestComponent();
testComponent.disabled = true; testComponent.disable = true;
const directive = new NodePermissionDirective(null, null, contentService, testComponent); const directive = new NodePermissionDirective(null, null, contentService, testComponent);
directive.nodes = <any> [{}, {}]; directive.nodes = <any> [{}, {}];
directive.updateElement(); directive.updateElement();
expect(testComponent.disabled).toBeFalsy(); expect(testComponent.disable).toBeFalsy();
}); });
}); });
}); });

View File

@@ -21,7 +21,7 @@ import { EXTENDIBLE_COMPONENT } from './../interface/injection.tokens';
import { AlfrescoContentService } from './../services/alfresco-content.service'; import { AlfrescoContentService } from './../services/alfresco-content.service';
export interface NodePermissionSubject { export interface NodePermissionSubject {
disabled: boolean; disable: boolean;
} }
@Directive({ @Directive({
@@ -74,7 +74,7 @@ export class NodePermissionDirective implements OnChanges, AfterViewInit {
private enable(): void { private enable(): void {
if (this.parentComponent) { if (this.parentComponent) {
this.parentComponent.disabled = false; this.parentComponent.disable = false;
} else { } else {
this.enableElement(); this.enableElement();
} }
@@ -82,7 +82,7 @@ export class NodePermissionDirective implements OnChanges, AfterViewInit {
private disable(): void { private disable(): void {
if (this.parentComponent) { if (this.parentComponent) {
this.parentComponent.disabled = true; this.parentComponent.disable = true;
} else { } else {
this.disableElement(); this.disableElement();
} }

View File

@@ -58,7 +58,7 @@ Contains components for handling uploads to Content Services.
| Name | Type | Default | Description | | Name | Type | Default | Description |
| --- | --- | --- | --- | | --- | --- | --- | --- |
| disabled | boolean | false | Toggle component disabled state | | disable | boolean | false | Toggle component disabled state |
| **(deprecated)** showNotificationBar | boolean | true | Hide/show notification bar. **Deprecated in 1.6.0: use UploadService events and NotificationService api instead.** | | **(deprecated)** showNotificationBar | boolean | true | Hide/show notification bar. **Deprecated in 1.6.0: use UploadService events and NotificationService api instead.** |
| uploadFolders | boolean | false | Allow/disallow upload folders (only for Chrome) | | uploadFolders | boolean | false | Allow/disallow upload folders (only for Chrome) |
| multipleFiles | boolean | false | Allow/disallow multiple files | | multipleFiles | boolean | false | Allow/disallow multiple files |

View File

@@ -54,8 +54,8 @@ export class UploadButtonComponent implements OnInit, OnChanges, NodePermissionS
@Input() @Input()
disableWithNoPermission: boolean = false; disableWithNoPermission: boolean = false;
@Input('disabled') @Input()
disabled: boolean = false; disable: boolean = false;
@Input() @Input()
uploadFolders: boolean = false; uploadFolders: boolean = false;
@@ -119,7 +119,7 @@ export class UploadButtonComponent implements OnInit, OnChanges, NodePermissionS
} }
isForceDisable(): boolean { isForceDisable(): boolean {
return this.disabled ? true : undefined; return this.disable ? true : undefined;
} }
/** @deprecated Deprecated in 1.8.0, use the button with combination of adf-node-permission directive */ /** @deprecated Deprecated in 1.8.0, use the button with combination of adf-node-permission directive */

View File

@@ -42,13 +42,13 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
@Input() @Input()
set enabled(enabled: boolean) { set enabled(enabled: boolean) {
console.warn('Deprecated: enabled input property should not be used for UploadDragAreaComponent. Please use disabled instead.'); console.warn('Deprecated: enabled input property should not be used for UploadDragAreaComponent. Please use disabled instead.');
this.disabled = !enabled; this.disable = !enabled;
} }
/** @deprecated Deprecated in favor of disabled input property */ /** @deprecated Deprecated in favor of disabled input property */
get enabled(): boolean { get enabled(): boolean {
console.warn('Deprecated: enabled input property should not be used for UploadDragAreaComponent. Please use disabled instead.'); console.warn('Deprecated: enabled input property should not be used for UploadDragAreaComponent. Please use disabled instead.');
return !this.disabled; return !this.disable;
} }
/** @deprecated Deprecated in 1.6.0, you can use UploadService events and NotificationService api instead. */ /** @deprecated Deprecated in 1.6.0, you can use UploadService events and NotificationService api instead. */
@@ -63,8 +63,8 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
@Input() @Input()
rootFolderId: string = '-root-'; rootFolderId: string = '-root-';
@Input('disabled') @Input()
disabled: boolean = false; disable: boolean = false;
@Input() @Input()
versioning: boolean = false; versioning: boolean = false;
@@ -86,7 +86,7 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
* @param {File[]} files - files dropped in the drag area. * @param {File[]} files - files dropped in the drag area.
*/ */
onFilesDropped(files: File[]): void { onFilesDropped(files: File[]): void {
if (!this.disabled && files.length) { if (!this.disable && files.length) {
const fileModels = files.map(file => new FileModel(file, { const fileModels = files.map(file => new FileModel(file, {
newVersion: this.versioning, newVersion: this.versioning,
path: '/', path: '/',
@@ -107,7 +107,7 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
* @param item - FileEntity * @param item - FileEntity
*/ */
onFilesEntityDropped(item: any): void { onFilesEntityDropped(item: any): void {
if (!this.disabled) { if (!this.disable) {
item.file((file: File) => { item.file((file: File) => {
const fileModel = new FileModel(file, { const fileModel = new FileModel(file, {
newVersion: this.versioning, newVersion: this.versioning,
@@ -129,7 +129,7 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
* @param folder - name of the dropped folder * @param folder - name of the dropped folder
*/ */
onFolderEntityDropped(folder: any): void { onFolderEntityDropped(folder: any): void {
if (!this.disabled && folder.isDirectory) { if (!this.disable && folder.isDirectory) {
FileUtils.flattern(folder).then(entries => { FileUtils.flattern(folder).then(entries => {
let files = entries.map(entry => { let files = entries.map(entry => {
return new FileModel(entry.file, { return new FileModel(entry.file, {
@@ -176,7 +176,7 @@ export class UploadDragAreaComponent implements NodePermissionSubject {
/** Returns true or false considering the component options and node permissions */ /** Returns true or false considering the component options and node permissions */
isDroppable(): boolean { isDroppable(): boolean {
return !this.disabled; return !this.disable;
} }
/** /**