Как правильно передать функции пользовательским директивам в Angular2 / Typescript? - PullRequest
0 голосов
/ 14 мая 2018

Я довольно новичок в Angular 2 . Я обновляю свое приложение из AngularJS, и сейчас я нахожусь на завершающей стадии разработки UI / UX. У меня есть последний вопрос, с которым мне нужна помощь. Заранее спасибо.

Текущий план

  1. У меня есть пользовательская директива TspFieldWidgetDirective , которая принимает несколько входов, один из которых является функцией @Input('onRefresh') public refresh: Function;.
  2. Элемент TspFieldWidgetDirective может использоваться внутри различных компонентов для изменения стандартных полей формы.
  3. Я использую текущую директиву в качестве поля выбора внутри шаблона TspHeaderComponent .
  4. Когда пользователь изменяет значение поля выбора, вызывается функция OnRefresh, чтобы обновить приложение и обновить представление для новой компании.

Что не работает

  1. Каждый раз, когда шаблон TspHeaderComponent загружается в браузер, он создает бесконечный цикл вызовов функции OnRefresh, onCompanyChange , когда это должно быть только вызывается один раз, когда поле выбора меняет значения.

  2. Когда я изменяю значение поля выбора из браузера (внутри шаблона TspFieldWidgetDirective - (change)="refresh({'id': fieldData[fieldKey]})", генерируется следующая ошибка. ERROR TypeError: _co.refresh is not a function

Обратите внимание

  1. Функциональность никогда не работала в Angular 5, она отлично работала в AngularJS.
  2. Все остальное, кроме передачи функции в качестве ввода в директиву, работает.

Фрагменты кода ниже:

tsp-header.component.ts

/*
    Method to change company of the company selector

    @param: id - string
    @return: none
 */
public onCompanyChange(id: string): void {
    if ((__env.debug && __env.verbose)) {
        console.log('Setting current_company_id to ' + id);
    }

    if (id !== undefined && id !== null)
    {
        // @TODO: Update user preference on server IF they have permission to change their default company
        this.cookies.set('preferences.current_company_id', id);
        this.core.app.current_user.preferences.current_company_id = id;
        this.core.initApp();

        this.router.navigate(['/app/dashboard']);
    }
}

tsp-header.html

<!-- company changer -->
<li>
  <tsp-field-widget 
      type="company-select" 
      [onRefresh]="onCompanyChange(id)" 
      [showAvatar]="true"
      [fieldData]="core.app.current_user.preferences"
      fieldKey="current_company_id" 
      [hideLabel]="true"
      [optionData]="core.app.session.base_companies">
  </tsp-field-widget>
</li>

tsp-field-widget.component.ts

// Component class
@Component({
  selector: 'tsp-field-widget',
  templateUrl: './templates/tsp-field-widget.html'
})
export class TspFieldWidgetComponent implements OnInit {
  public lang:any; // for i18n

  @Input() public type: string; // the field type
  @Input() public isRequired: boolean; // is the field required
  @Input() public isReadonly: boolean;

  @Input() public index: any; // index of ng-repeat
  @Input() public filterBy: any; // used in conjunction with ng-repeat
  @Input() public orderBy: any; // used in conjunction with ng-repeat
  @Input() public fieldData: any; // the record of ng-repeat
  @Input() public fieldKey: string; // the index of record - record[fieldKey]
  @Input() public placeVal: string; // the placeholder value of the field, usually used for selects and other dropdowns
  @Input() public pattern: string; // used for ng-pattern
  @Input() public prefix: string; // Text to display before title listings

  @Input() public recordId: any; // the ID of the record
  @Input() public label: string; // the label for the field for <label> tag
  @Input() public suffix: string; // sub label, usually placed below some label or title
  @Input() public optionData: any[]; // array of data used to populate selects or to store data values
  @Input() public showAvatar: boolean; // show the fields avatar
  @Input() public hideLabel: boolean; // show the fields <label>
  @Input() public showAdd: boolean; // show the add button (to add new record)
  @Input() public showTitle: boolean; // avatar type: show the name of the user
  @Input() public showDesc: boolean; // avatar type: show the name of the user
  // @Input() public isHighlighted:boolean; // default type: highlight text
  @Input() public numRows: string; // textarea: the number of rows of the text area
  @Input() public idKey: string; // select: the default ID key for option data - defaults to _id
  @Input() public titleKey: string; // select: the default title key for option data - defaults to title
  @Input() public descKey: string; // select: the default description key for option data - defaults to description

  @Input() public sliderType: string; // percent, amount or default, slider type
  @Input() public sliderMinValue: string; // slider type
  @Input() public sliderMaxValue: string; // slider type
  @Input() public sliderStepValue: string; // slider type
  @Input() public sliderOrientation: string; // slider type

  @Input() public dzOptions: any; // dropzone options
  @Input() public dzCallbacks: any; // dropzone callbacks
  @Input() public dzMethods: any; // dropzone methods

  @Input() public decimalPlaces: string; // tspFormat type

  @Input() public locale: string; // for dates and currency
  @Input() public format: string; // for dates
  @Input() public formatHours: string; // for dates
  @Input() public intervalMins: number; // for dates
  @Input() public owner: string; // used for module windows to determine the type of record to add

  @Input('onAdd') public add: Function;
  @Input('onEdit') public edit: Function;
  @Input('onToggle') public toggle: Function;
  @Input('onDelete') public delete: Function;
  @Input('onRefresh') public refresh: Function;

  constructor(private el: ElementRef,
    private cookies: TspCookiesService,
    public core: TspCoreService,
    private object: TspObjectService,
    public date: TspDateService) {
    this.lang = core.lang;
    this.date = date;
  }
}

tsp-field-widget.html

<div *ngIf="type=='company-select'">
  <select class="company-select" 
      class="form-control" 
      [(ngModel)]="fieldData[fieldKey]" 
      (change)="refresh({'id': fieldData[fieldKey]})" 
      data-parsley-trigger="change">
    <option [selected]="x[idKey] === fieldData[fieldKey]" *ngFor="let x of optionData" [ngValue]="x[idKey]">
      {{x[titleKey]}}
    </option>
  </select>
</div>

Ответы [ 2 ]

0 голосов
/ 14 мая 2018

Полное исправление

tsp-field-widget.component.ts - Изменены все @Inputs типа Function на @Ouput и инициализированы как EventEmitters.Заменены все префиксы on на evt, так как on не разрешен в качестве префикса для @Output.Добавлены четыре новых метода, которые будут вызываться после запуска событий.Каждый из новых методов должен иметь интерфейс, назначенный для аргумента args для согласованности

// Component class
@Component({
  selector: 'tsp-field-widget',
  templateUrl: './templates/tsp-field-widget.html'
})
export class TspFieldWidgetComponent implements OnInit {
  public lang:any; // for i18n

  @Input() public type: string; // the field type
  @Input() public isRequired: boolean; // is the field required
  @Input() public isReadonly: boolean;

  @Input() public index: any; // index of ng-repeat
  @Input() public filterBy: any; // used in conjunction with ng-repeat
  @Input() public orderBy: any; // used in conjunction with ng-repeat
  @Input() public fieldData: any; // the record of ng-repeat
  @Input() public fieldKey: string; // the index of record - record[fieldKey]
  @Input() public placeVal: string; // the placeholder value of the field, usually used for selects and other dropdowns
  @Input() public pattern: string; // used for ng-pattern
  @Input() public prefix: string; // Text to display before title listings

  @Input() public recordId: any; // the ID of the record
  @Input() public label: string; // the label for the field for <label> tag
  @Input() public suffix: string; // sub label, usually placed below some label or title
  @Input() public optionData: any[]; // array of data used to populate selects or to store data values
  @Input() public showAvatar: boolean; // show the fields avatar
  @Input() public hideLabel: boolean; // show the fields <label>
  @Input() public showAdd: boolean; // show the add button (to add new record)
  @Input() public showTitle: boolean; // avatar type: show the name of the user
  @Input() public showDesc: boolean; // avatar type: show the name of the user
  // @Input() public isHighlighted:boolean; // default type: highlight text
  @Input() public numRows: string; // textarea: the number of rows of the text area
  @Input() public idKey: string; // select: the default ID key for option data - defaults to _id
  @Input() public titleKey: string; // select: the default title key for option data - defaults to title
  @Input() public descKey: string; // select: the default description key for option data - defaults to description

  @Input() public sliderType: string; // percent, amount or default, slider type
  @Input() public sliderMinValue: string; // slider type
  @Input() public sliderMaxValue: string; // slider type
  @Input() public sliderStepValue: string; // slider type
  @Input() public sliderOrientation: string; // slider type

  @Input() public dzOptions: any; // dropzone options
  @Input() public dzCallbacks: any; // dropzone callbacks
  @Input() public dzMethods: any; // dropzone methods

  @Input() public decimalPlaces: string; // tspFormat type

  @Input() public locale: string; // for dates and currency
  @Input() public format: string; // for dates
  @Input() public formatHours: string; // for dates
  @Input() public intervalMins: number; // for dates
  @Input() public owner: string; // used for module windows to determine the type of record to add

  @Output() public evtAdd = new EventEmitter();
  @Output() public evtEdit = new EventEmitter();
  @Output() public evtToggle = new EventEmitter();
  @Output() public evtDelete = new EventEmitter();
  @Output() public evtRefresh = new EventEmitter();

  constructor(private el: ElementRef,
    private cookies: TspCookiesService,
    public core: TspCoreService,
    private object: TspObjectService,
    public date: TspDateService) {
    this.lang = core.lang;
    this.date = date;
  }
  add(args: IAdd){
    this.evtAdd.emit(args);
  }
  edit(args: IEdit){
    this.evtEdit.emit(args);
  }
  toggle(args: IToggle){
    this.evtToggle.emit(args);
  }
  delete(args: IDelete){
    this.evtDelete.emit(args);
  }
  refresh(args: IRefresh){
    this.evtRefresh.emit(args);
  }
}

tsp-field-widget.html - Изменения не требуются

tsp-header.component.ts - Обновлено для передачи объекта, который содержит значения, необходимые для функции.

public onCompanyChange(args: IRefresh):void {
    if (args !== undefined){
        if (args.id !== undefined && args.id !== null)
        {
            if ((__env.debug && __env.verbose)) {
                console.log('Setting current_company_id to ' + args.id);
            }

            // @TODO: Update user preference on server IF they have permission to change their default company
            this.cookies.set('preferences.current_company_id', args.id);
            this.core.app.current_user.preferences.current_company_id = args.id;
            this.core.initApp();

            this.router.navigate(['/app/dashboard']);
        }
    }
}

tsp-header.html - Переименован атрибут onRefresh в evtRefresh,Чтобы предотвратить бесконечный цикл, я обернул атрибут evtRefresh круглыми скобками, а не скобками, чтобы обозначить, что атрибут является событием, а НЕ объектом.Также аргумент функции всегда должен быть $event.

<!-- company changer -->
<li>
  <tsp-field-widget 
      type="company-select" 
      (evtRefresh)="onCompanyChange($event)" 
      [showAvatar]="true"
      [fieldData]="core.app.current_user.preferences"
      fieldKey="current_company_id" 
      [hideLabel]="true"
      [optionData]="core.app.session.base_companies">
  </tsp-field-widget>
</li>
0 голосов
/ 14 мая 2018

Обнаружение изменений Angular уже срабатывает во время инициализации вида.Но на данный момент refresh () все еще может быть нулевым / неопределенным.

Я рекомендую не вызывать метод refresh () напрямую, а использовать метод-обертку, который может проверить, передан ли в функцию refresh ().

Что-то подобное в вашем tsp-field-widget.html :

(change)="doRefresh({'id': fieldData[fieldKey]})"

и это в вашем tsp-field-widget.component.ts

private doRefresh(object: any): void {

    if(refresh) {
       refresh(object);
    }

}

Таким образом, вы можетеубедитесь, что обновление вызывается только в том случае, если оно уже существует.

...