Code cleanup.

ReCodEx · Jan 9, 2022 · e31fdc3 · e31fdc3
1 parent cb4b4b7
commit e31fdc3
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 198 deletions.
diff --git a/src/components/Pipelines/PipelinesSimpleList/PipelinesSimpleList.js b/src/components/Pipelines/PipelinesSimpleList/PipelinesSimpleList.js
diff --git a/src/components/Pipelines/PipelinesSimpleList/index.js b/src/components/Pipelines/PipelinesSimpleList/index.js
diff --git a/src/components/Pipelines/PipelinesSimpleListItem/PipelinesSimpleListItem.js b/src/components/Pipelines/PipelinesSimpleListItem/PipelinesSimpleListItem.js
diff --git a/src/components/Pipelines/PipelinesSimpleListItem/index.js b/src/components/Pipelines/PipelinesSimpleListItem/index.js
diff --git a/src/pages/EditExerciseConfig/EditExerciseConfig.js b/src/pages/EditExerciseConfig/EditExerciseConfig.js
@@ -17,7 +17,6 @@ import EditExerciseAdvancedConfigForm from '../../components/forms/EditExerciseA
 import EditEnvironmentSimpleForm from '../../components/forms/EditEnvironmentSimpleForm';
 import EditEnvironmentConfigForm from '../../components/forms/EditEnvironmentConfigForm';
 import EditExercisePipelinesForm from '../../components/forms/EditExercisePipelinesForm/EditExercisePipelinesForm';
-// import PipelinesSimpleList from '../../components/Pipelines/PipelinesSimpleList';
 import ExerciseCallouts, { exerciseCalloutsAreVisible } from '../../components/Exercises/ExerciseCallouts';
 import ExerciseConfigTypeButton from '../../components/buttons/ExerciseConfigTypeButton';
 import { InfoIcon, TestsIcon } from '../../components/icons';
@@ -42,14 +41,6 @@ import {
 } from '../../redux/selectors/supplementaryFiles';
 import { isLoadingState } from '../../redux/helpers/resourceManager/status';
 
-/*
-import {
-  deletePipeline,
-  fetchExercisePipelines
-} from '../../redux/modules/pipelines';
-import { exercisePipelinesSelector } from '../../redux/selectors/pipelines';
-*/
-
 import withLinks from '../../helpers/withLinks';
 import { exerciseEnvironmentConfigSelector } from '../../redux/selectors/exerciseEnvironmentConfigs';
 import {
@@ -279,14 +270,10 @@ class EditExerciseConfig extends Component {
       exerciseScoreConfig,
       exerciseTests,
       pipelines,
-      // exercisePipelines,
       environmentsWithEntryPoints,
       pipelinesVariables,
       supplementaryFiles,
       supplementaryFilesStatus,
-      // links: {
-      // PIPELINE_EDIT_URI_FACTORY
-      // },
     } = this.props;
 
     return (
@@ -482,81 +469,6 @@ class EditExerciseConfig extends Component {
                                 </p>
                               </Callout>
                             )}
-
-                            {/* exercise.configurationType !== 'simpleExerciseConfig' &&
-                  <Box
-                    title={
-                      <FormattedMessage
-                        id="app.exercise.exercisePipelines"
-                        defaultMessage="Exercise Pipelines"
-                      />
-                    }
-                    footer={
-                      <p className="text-center">
-                        <Button
-                          variant="success"
-                          size="sm"
-                          onClick={this.createExercisePipeline}
-                        >
-                          <AddIcon gapRight />
-                          <FormattedMessage
-                            id="app.exercise.createPipeline"
-                            defaultMessage="Add exercise pipeline"
-                          />
-                        </Button>
-                      </p>
-                    }
-                    isOpen
-                  >
-                    <ResourceRenderer
-                      resource={exercisePipelines.toArray()}
-                      returnAsArray={true}
-                    >
-                      {pipelines =>
-                        <PipelinesSimpleList
-                          pipelines={pipelines}
-                          createActions={pipelineId =>
-                            <div>
-                              <Link
-                                to={PIPELINE_EDIT_URI_FACTORY(pipelineId)}
-                              >
-                                <Button
-                                  size="xs"
-                                  variant="warning"
-                                >
-                                  <EditIcon gapRight />
-                                  <FormattedMessage
-                                    id="generic.edit"
-                                    defaultMessage="Edit"
-                                  />
-                                </Button>
-                              </Link>
-                              <Confirm
-                                id={pipelineId}
-                                onConfirmed={() => deletePipeline(pipelineId)}
-                                question={
-                                  <FormattedMessage
-                                    id="app.pipeline.deleteConfirm"
-                                    defaultMessage="Are you sure you want to delete the pipeline? This cannot be undone."
-                                  />
-                                }
-                              >
-                                <Button
-                                  size="xs"
-                                  variant="danger"
-                                >
-                                  <DeleteIcon gapRight />
-                                  <FormattedMessage
-                                    id="generic.delete"
-                                    defaultMessage="Delete"
-                                  />
-                                </Button>
-                              </Confirm>
-                            </div>}
-                        />}
-                    </ResourceRenderer>
-                  </Box>
-                */}
                           </Col>
                         </Row>
                       )}
@@ -724,7 +636,6 @@ export default withLinks(
         exerciseScoreConfig: exerciseScoreConfigSelector(state, exerciseId),
         exerciseTests: exerciseTestsSelector(exerciseId)(state),
         pipelines: pipelinesSelector(state),
-        //        exercisePipelines: exercisePipelinesSelector(exerciseId)(state),
         environmentsWithEntryPoints: getPipelinesEnvironmentsWhichHasEntryPoint(state),
         pipelinesVariables: getExercisePielinesVariablesJS(exerciseId)(state),
         supplementaryFiles: getSupplementaryFilesForExercise(exerciseId)(state),

diff --git a/src/pages/Exercise/Exercise.js b/src/pages/Exercise/Exercise.js
@@ -77,7 +77,6 @@ class Exercise extends Component {
       dispatch(fetchRuntimeEnvironments()),
       dispatch(fetchReferenceSolutions(exerciseId)),
       dispatch(fetchHardwareGroups()),
-      //      dispatch(fetchExercisePipelines(exerciseId)), // TODO - awaiting modification (many-to-many relation with exercises)
     ]);
 
   componentDidMount() {

diff --git a/src/redux/modules/pipelines.js b/src/redux/modules/pipelines.js
@@ -89,27 +89,6 @@ const reducer = handleActions(
     [pipelineFilesActionTypes.ADD_FILES_FULFILLED]: (state, { payload: files, meta: { pipelineId } }) =>
       state.hasIn(['resources', pipelineId]) ? updateFiles(state, pipelineId, files, 'supplementaryFilesIds') : state,
 
-    /*
-      [additionalActionTypes.FORK_PIPELINE_PENDING]: (state, { meta: { id, forkId } }) =>
-      state.updateIn(['resources', id], pipeline => {
-        if (!pipeline.has('forks')) {
-          pipeline = pipeline.set('forks', new Map());
-        }
-
-        return pipeline.update('forks', forks => forks.set(forkId, { status: forkStatuses.PENDING }));
-      }),
-
-    [additionalActionTypes.FORK_PIPELINE_REJECTED]: (state, { meta: { id, forkId } }) =>
-      state.setIn(['resources', id, 'forks', forkId], {
-        status: forkStatuses.REJECTED,
-      }),
-
-    [additionalActionTypes.FORK_PIPELINE_FULFILLED]: (state, { payload: { id: pipelineId }, meta: { id, forkId } }) =>
-      state.setIn(['resources', id, 'forks', forkId], {
-        status: forkStatuses.FULFILLED,
-        pipelineId,
-      }),
-*/
     [additionalActionTypes.SET_ENVIRONMENTS_FULFILLED]: (state, { payload }) =>
       state.setIn(
         ['resources', payload.id],