r/p5.js-web-editor
1
0
Fork 0
Code Issues 1 Pull requests Releases 1 Wiki Activity

Move handleUploadClick from inline function, prevent user from uploading to folder if not authenticated

Browse source
This commit is contained in:
Cassie Tarakajian 2020-04-16 16:08:51 -04:00
parent 9185447168
commit 41637d2837
2 changed files with 22 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
client/components/__test__/FileNode.test.jsx
View file

@ -24,6 +24,7 @@ describe('<FileNode />', () => {
showFolderChildren: jest.fn(),
hideFolderChildren: jest.fn(),
canEdit: true,
authenticated: false
};
component = shallow(<FileNode {...props} />);
});

18
client/modules/IDE/components/FileNode.jsx
View file

@ -87,6 +87,11 @@ export class FileNode extends React.Component {
setTimeout(() => this.hideFileOptions(), 0);
}
handleClickUploadFile = () => {
this.props.openUploadFileModal(this.props.id);
setTimeout(this.hideFileOptions, 0);
}
handleClickDelete = () => {
if (window.confirm(`Are you sure you want to delete ${this.props.name}?`)) {
this.setState({ isDeleting: true });
@ -252,19 +257,18 @@ export class FileNode extends React.Component {
Create file
</button>
</li>
{ this.props.authenticated &&
<li>
<button
aria-label="upload file"
onClick={() => {
this.props.openUploadFileModal(this.props.id);
setTimeout(this.hideFileOptions, 0);
}}
onClick={this.handleClickUploadFile}
onBlur={this.onBlurComponent}
onFocus={this.onFocusComponent}
>
Upload file
</button>
</li>
}
</React.Fragment>
}
<li>
@ -318,7 +322,8 @@ FileNode.propTypes = {
showFolderChildren: PropTypes.func.isRequired,
hideFolderChildren: PropTypes.func.isRequired,
canEdit: PropTypes.bool.isRequired,
openUploadFileModal: PropTypes.func.isRequired
openUploadFileModal: PropTypes.func.isRequired,
authenticated: PropTypes.bool.isRequired
};
FileNode.defaultProps = {
@ -329,7 +334,8 @@ FileNode.defaultProps = {
function mapStateToProps(state, ownProps) {
// this is a hack, state is updated before ownProps
return state.files.find(file => file.id === ownProps.id) || { name: 'test', fileType: 'file' };
const fileNode = state.files.find(file => file.id === ownProps.id) || { name: 'test', fileType: 'file' };
return Object.assign({}, fileNode, { authenticated: state.user.authenticated });
}
function mapDispatchToProps(dispatch) {