scratchfoundation/scratch-www
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-www.git synced 2025-02-17 00:21:20 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

further debugging addtostudio modal

Browse source
This commit is contained in:
Ben Wheeler 2018-06-19 11:26:02 -04:00
parent 61dda43b8f
commit 1b874b748c
1 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/components/modal/addtostudio/modal.jsx
View file

@ -64,7 +64,6 @@ class AddToStudioModal extends React.Component {
}
componentWillReceiveProps(nextProps) {
debugger;
this.updateOnOrDirty(nextProps.projectStudios, nextProps.myStudios);
}
@ -74,6 +73,7 @@ class AddToStudioModal extends React.Component {
// them too; otherwise we might retain a dirty change for a studio
// we no longer have permission for. In theory.
debugger;
let onOrDirty = this.state.onOrDirty;
projectStudios.forEach((studio) => {
onOrDirty[studio.id] = {added: true, dirty: false};
@ -121,7 +121,7 @@ class AddToStudioModal extends React.Component {
handleSubmit (formData) {
// NOTE: ignoring formData for now...
this.setState({waiting: true});
const onOrDirty = this.state.onOrDirty;
let onOrDirty = this.state.onOrDirty;
const studiosToAdd = Object.keys(onOrDirty)
.reduce(function(accumulator, key) {
if (onOrDirty[key].dirty === true &&
@ -138,11 +138,17 @@ class AddToStudioModal extends React.Component {
}
return accumulator;
}, []);
onOrDirty = {};
// When this modal is opened, and isOpen becomes true,
// onOrDirty should start with a clean slate
// NOTE: this doesn't seem to be working
this.setState({onOrDirty: {testkey3: 'testval3'}});
// NOTE: this doesn't seem to be working:
this.setState({ onOrDirty: undefined });
// what's weird is, it's not an issue with "this" being invalid,
// because this works fine:
this.setState({testkey: 'testval'});
debugger;
this.props.onAddToStudio(studiosToAdd, studiosToDelete, err => {
if (err) log.error(err);