ReferenceError when reorganizing cloud functions

Question

I'm following the instructions on https://firebase.google.com/docs/functions/organize-functions to reorganize my cloud functions.

But I'm struggling to reuse some of the global const defined in the index.js file.

// index.js
const functions = require('firebase-functions')
const admin = require('firebase-admin')
admin.initializeApp()

const db = admin.firestore()
db.settings({ ignoreUndefinedProperties: true })

exports.choice = require('./choice')
...

// choice.js
const functions = require('firebase-functions')

exports.redirection = functions
    .region('europe-west1')
    .https.onRequest(async (req, res) => {
        try {
            const doc = await db.doc(`stbk${req.params[0]}`).get()
...

Got the following error: > ReferenceError: db is not defined

Any help would be greatly appreciated !

UPDATE

I've done the following:

// global.js
const admin = require('firebase-admin')
admin.initializeApp()

const db = admin.firestore()
db.settings({ ignoreUndefinedProperties: true })

module.exports = { admin, db }

// choice.js
const functions = require('firebase-functions')
const global = require('./global')

exports.redirection = functions
    .region('europe-west1')
    .https.onRequest(async (req, res) => {
        try {
            const doc = await global.db.doc(`stbk${req.params[0]}`).get()

Is it a valid solution?

Nibrass H Nibrass H · Accepted Answer · 2020-08-05T14:37:21

You are getting the error because you are not initializing the Firestore database correctly. You can use the following code:

Index.js

const functions = require('firebase-functions');
const admin = require('firebase-admin');
admin.initializeApp(functions.config().firestore);

const firestoreDB = admin.firestore()

exports.choice = require('./choice')
module.exports = { db }

Choice.js


const functions = require('firebase-functions')
const global = require('./index')

exports.redirection = functions
    .region('europe-west1')
    .https.onRequest(async (req, res) => {
        try {
            const doc = await global.doc(`stbk${req.params[0]}`).get()

I think only the index.js and choice.js are sufficient. Why do you want to create an another .js called global? If you want to separate as mentioned above in global.js, then your code seems valid to me.

Let me know if you have any other questions.

ReferenceError when reorganizing cloud functions

2 Answers