singleDocData option seems works only with .getById

[Merkur39]

Avec une requête comme ci-dessous :

export const buildAppSettingsQuery = () => ({
  definition: () => Q(SETTINGS_DOCTYPE),
  options: {
    singleDocData: true,
    as: `${SETTINGS_DOCTYPE}`,
    fetchPolicy: defaultFetchPolicy
  }
})

Nous obtenons toujours le résultat suivant : { data: [{ _id: 'foo', ... }], ... }

La solution pour que l'option singleDocData soit respectée est de passer par la méthode getById, soit :

export const buildAppSettingsQuery = () => ({
  definition: () => Q(SETTINGS_DOCTYPE).getById('configuration'),
  options: {
    singleDocData: true,
    as: `${SETTINGS_DOCTYPE}`,
    fetchPolicy: defaultFetchPolicy
  }
})

Et ainsi nous obtenons bien : { data: { _id: 'foo', ... }, ... }

[paultranvan]

This is because of

cozy-client/packages/cozy-client/src/CozyClient.js

Lines 1336 to 1337 in 4cf0e80

	const isSingleDocQuery =
	queryResults.definition && queryResults.definition.id

, which is indeed a bit misleading

But since there is no .limitBy(1) in your example, what would be the expected behavior if there is actually more than one doc? Return the first one? Throw an error? Ignore the singleDocData?

[Merkur39]

Ok, I missed that info

1. I would say return the most recent, we should still have the information via the count if there is more than one.

2. As for the .limitBy(1), it should do it implicitly (I think I saw an issue at this subject)

The first point is debatable, maybe limiting to the use of limitBy is better 🤷