Skip to content

Commit

Permalink
fix: if sim doesn't have translated file, set shared strings to empty
Browse files Browse the repository at this point in the history
For #342.

Part of phetsims/qa#869.
  • Loading branch information
liammulh committed Dec 21, 2022
1 parent e7cee15 commit 85f046c
Showing 1 changed file with 19 additions and 3 deletions.
22 changes: 19 additions & 3 deletions src/server/translationApi/getSharedTranslationFormData.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,19 @@ const getSharedTranslationFormData = async (
for ( const sim of sharedSims ) {
const simFileRes = await axios.get( getStringFileUrl( sim ) );
englishStringFiles.push( simFileRes.data );
const translatedFileRes = await axios.get( getTranslatedStringFile( sim, locale ) );
if ( translatedFileRes.status >= 200 && translatedFileRes.status <= 300 ) {
translatedStringFiles.push( translatedFileRes.data );
try {
const translatedFileRes = await axios.get( getTranslatedStringFile( sim, locale ) );
if ( translatedFileRes.status >= 200 && translatedFileRes.status <= 300 ) {
translatedStringFiles.push( translatedFileRes.data );
}
}
catch( e ) {
if ( e.response.status === 404 ) {
logger.verbose( `translation file for ${simName} doesn't exist; setting empty strings for ${simName}` );
}
else {
logger.error( e );
}
}
}

Expand Down Expand Up @@ -79,6 +89,12 @@ const getSharedTranslationFormData = async (
translated: translatedKeysAndValues[ stringKey ]
};
}
else if ( englishKeysAndValues[ stringKey ] && !translatedKeysAndValues[ stringKey ] ) {
sharedTranslationFormData[ stringKeyWithoutDots ] = {
english: englishKeysAndValues[ stringKey ],
translated: ''
};
}
}
}
catch( e ) {
Expand Down

0 comments on commit 85f046c

Please sign in to comment.