refactor: Remove unused defaultToAdvanced and getBetaParsedOLXData (#1753)

edx-platform would pass a default_to_advanced flag in through the REST
API, depending on the value of a waffle flag. The flag did not actually
cause anything to default to advanced. What it actually did was switch
from getParsedOLXData to getParsedBetaOLXData. However, getBetaOLXParser
was never implemented--it just logs a console warning and return
getOLXParser.

We remove this unused flag and unused function.
The underlying default_to_advanced API flag and the backing waffle flag
will be removed from edx-platform in a separate PR.
This commit is contained in:
Kyle McCormick
2025-03-20 15:49:36 -04:00
committed by GitHub
parent 27c4eec746
commit 9e65424ca6
4 changed files with 1 additions and 17 deletions

View File

@@ -23,7 +23,6 @@ export const onSelect = ({
attempts_before_showanswer_button: 0,
show_reset_button: null,
showanswer: null,
defaultToAdvanced: false,
},
defaultSettings: snakeCaseKeys(defaultSettings),
});

View File

@@ -55,7 +55,6 @@ describe('SelectTypeModal hooks', () => {
attempts_before_showanswer_button: 0,
show_reset_button: null,
showanswer: null,
defaultToAdvanced: false,
},
defaultSettings: mockDefaultSettings,
});

View File

@@ -753,13 +753,4 @@ export class OLXParser {
groupFeedbackList,
};
}
getBetaParsedOLXData() {
/* TODO: Replace olxParser.getParsedOLXData() with new parser function
* and remove console.log()
*/
// eslint-disable-next-line no-console
console.log('Should default to the advanced editor');
return this.getParsedOLXData();
}
}

View File

@@ -34,14 +34,9 @@ export const isBlankProblem = ({ rawOLX }) => {
export const getDataFromOlx = ({ rawOLX, rawSettings, defaultSettings }) => {
let olxParser;
let parsedProblem;
const { default_to_advanced: defaultToAdvanced } = rawSettings;
try {
olxParser = new OLXParser(rawOLX);
if (defaultToAdvanced) {
parsedProblem = olxParser.getBetaParsedOLXData();
} else {
parsedProblem = olxParser.getParsedOLXData();
}
parsedProblem = olxParser.getParsedOLXData();
} catch (error) {
// eslint-disable-next-line no-console
console.error('The Problem Could Not Be Parsed from OLX. redirecting to Advanced editor.', error);