player page: goback only if backstack available, go to homepage if not

fix invalid api request when creating new actor
This commit is contained in:
lukas 2021-08-28 22:21:51 +02:00
parent 032b90a93d
commit f8dbadc45b
2 changed files with 10 additions and 5 deletions

View File

@ -22,7 +22,7 @@ class NewActorPopup extends React.Component<NewActorPopupProps> {
} }
export class NewActorPopupContent extends React.Component<NewActorPopupProps> { export class NewActorPopupContent extends React.Component<NewActorPopupProps> {
value: string | undefined; nameValue: string | undefined;
render(): JSX.Element { render(): JSX.Element {
return ( return (
@ -32,7 +32,7 @@ export class NewActorPopupContent extends React.Component<NewActorPopupProps> {
type='text' type='text'
placeholder='Actor Name' placeholder='Actor Name'
onChange={(v): void => { onChange={(v): void => {
this.value = v.target.value; this.nameValue = v.target.value;
}} }}
/> />
</div> </div>
@ -48,11 +48,11 @@ export class NewActorPopupContent extends React.Component<NewActorPopupProps> {
*/ */
storeselection(): void { storeselection(): void {
// check if user typed in name // check if user typed in name
if (this.value === '' || this.value === undefined) { if (this.nameValue === '' || this.nameValue === undefined) {
return; return;
} }
callAPI(APINode.Actor, {action: 'createActor', actorname: this.value}, (result: GeneralSuccess) => { callAPI(APINode.Actor, {action: 'createActor', ActorName: this.nameValue}, (result: GeneralSuccess) => {
if (result.result !== 'success') { if (result.result !== 'success') {
console.log('error occured while writing to db -- todo error handling'); console.log('error occured while writing to db -- todo error handling');
console.log(result.result); console.log(result.result);

View File

@ -314,7 +314,12 @@ export class Player extends React.Component<Props, mystate> {
* calls callback to viewbinding to show previous page agains * calls callback to viewbinding to show previous page agains
*/ */
closebtn(): void { closebtn(): void {
this.props.history.goBack(); const hist = this.props.history;
if (hist.length > 1) {
this.props.history.goBack();
} else {
hist.push('/');
}
} }
/** /**