TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-12-27 12:53:13 +03:00
Code Issues Projects Releases Wiki Activity

Changed Error to IncorrectUsageError in package-json

Browse Source 
no issue

- `Error` is very generic for this case and `IncorrectUsageError`
  will populate the resulting error with the correct error code
- the `message` was pulled out to its own statement so we can avoid long
  lines
This commit is contained in:
Daniel Lockyer 2021-05-06 13:22:59 +01:00
parent eea93d55f4
commit 95cfa97747
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ghost/package-json/lib/package-json.js
View File

@ -84,7 +84,8 @@ module.exports = class PackageJson {
try {
source = await fs.readFile(path);
} catch (readError) {
const err = new Error(this.i18n.t('errors.utils.parsepackagejson.couldNotReadPackage'));
const err = new errors.IncorrectUsageError();
err.message = this.i18n.t('errors.utils.parsepackagejson.couldNotReadPackage');
err.context = path;
err.err = readError;
@ -94,7 +95,8 @@ module.exports = class PackageJson {
try {
json = JSON.parse(source);
} catch (parseError) {
const err = new Error(this.i18n.t('errors.utils.parsepackagejson.themeFileIsMalformed'));
const err = new errors.IncorrectUsageError();
err.message = this.i18n.t('errors.utils.parsepackagejson.themeFileIsMalformed');
err.context = path;
err.err = parseError;
err.help = this.i18n.t('errors.utils.parsepackagejson.willBeRequired', {url: 'https://ghost.org/docs/themes/'});
@ -105,7 +107,8 @@ module.exports = class PackageJson {
const hasRequiredKeys = json.name && json.version;
if (!hasRequiredKeys) {
const err = new Error(this.i18n.t('errors.utils.parsepackagejson.nameOrVersionMissing'));
const err = new errors.IncorrectUsageError();
err.message = this.i18n.t('errors.utils.parsepackagejson.nameOrVersionMissing');
err.context = path;
err.help = this.i18n.t('errors.utils.parsepackagejson.willBeRequired', {url: 'https://ghost.org/docs/themes/'});