fix: no request sent when disableStringsCache:true

[sapjax]

In the origin code, if disableStringsCache is set to true, the getFileTranslations will never be called.

[yevheniyJ]

cache stores promises, that's why we have await 2 lines above when extracting it from cache. This is needed to prevent "double" loading. Please update it to:

let content;
if (this.disableStringsCache) {
  content = await this.getFileTranslations(filePath);
} else {
  if (!this.stringsCache[filePath]) {
     this.stringsCache[filePath] = this.getFileTranslations(filePath);
  }
  content = await this.stringsCache[filePath];
}

[sapjax]

Changed as your comment.
But I don't understand why we need to cache promises instead resolved data, what is the "double" loading case.
The request is already sent, just when to resolve it from promise, should be no difference indeed.