I'm trying to assign the data collected from a firestore document to a variable with an Observable type initialised before the constructor.
I've got the data from the collection by passing in a dynamic invoiceId string to the .doc() search, and I can assign the data to a local variable (shown below), but when trying to assign it to this.invoice, I get the following error:
Uncaught (in promise): TypeError: Cannot set property 'invoice' of undefined
-
Component:
import { Component, OnInit, Input } from '@angular/core';
import { ActivatedRoute } from '@angular/router';
import { Observable } from 'rxjs/Observable';
import { AngularFireDatabase } from 'angularfire2/database';
import { AngularFirestore, AngularFirestoreCollection, AngularFirestoreDocument } from 'angularfire2/firestore';
import { AuthService } from '../../services/auth.service';
import { Invoice } from '../invoiceModel';
@Component({
selector: 'app-view-invoice',
templateUrl: './view-invoice.component.html',
styleUrls: ['./view-invoice.component.scss']
})
export class ViewInvoiceComponent implements OnInit {
userId: string;
invoiceId: any;
invoicesCollection: AngularFirestoreCollection<Invoice>;
invoices: Observable<Invoice[]>;
invoice: Observable<Invoice>;
constructor(private authService: AuthService, private db: AngularFirestore, private route: ActivatedRoute) {
this.userId = this.authService.user.uid;
this.route.params.subscribe(params => {
this.invoiceId = params.id;
})
this.invoicesCollection = this.db.collection('/invoices');
this.invoices = this.invoicesCollection.snapshotChanges().map(changes => {
return changes.map(a => {
const data = a.payload.doc.data() as Invoice;
data.id = a.payload.doc.id;
return data;
})
})
}
ngOnInit() {
this.getInvoice();
}
getInvoice() {
var docref = this.db.collection('/users').doc(this.authService.user.uid).collection('/invoices').doc(this.invoiceId);
docref.ref.get()
.then(function(doc) {
if (doc.exists) {
var invoice = doc.data(); <------WORKS
// this.invoice = doc.data(); <------DOESN'T WORK
console.log('Invoice data: ', doc.data());
} else {
console.error('No matching invoice found');
}
})
}
}
this
. as ingetInvoice() { const _that=this;
and then withindoc.exists
try_that.invoice = doc.data()
– Guruprasad J Raothis
would not be actuallyComponent
since it is within a promise. That's my guess.. So you are able to set it with_that
? – Guruprasad J Raothis
. But the changes you make with_that
actually affects the original variable.. :) – Guruprasad J Rao