Fix page reload navigation and add more debugging
This commit is contained in:
parent
bbb727bd38
commit
0c93860816
@ -192,6 +192,8 @@ export class AppComponent implements OnInit {
|
||||
const id = node.data.id;
|
||||
const nodeId = node.data.node_id;
|
||||
if ( !node.data.virtual ) {
|
||||
debug('Navigating editor to node:', {id, nodeId});
|
||||
|
||||
this.currentPageId = id;
|
||||
this.router.navigate(['/editor', { id, ...(nodeId ? { node_id: nodeId } : {}) }]);
|
||||
}
|
||||
@ -608,6 +610,12 @@ export class AppComponent implements OnInit {
|
||||
});
|
||||
|
||||
this.navService.navigationRequest$.subscribe(pageId => {
|
||||
debug('Page navigation request: ', {pageId});
|
||||
if ( !pageId ) {
|
||||
debug('Empty page ID. Will not navigate.');
|
||||
return;
|
||||
}
|
||||
|
||||
this.currentPageId = pageId;
|
||||
this.router.navigate(['/editor', { id: pageId }]);
|
||||
});
|
||||
|
@ -43,6 +43,8 @@ export class EditorPage implements OnInit {
|
||||
protected modals: ModalController,
|
||||
public readonly editorService: EditorService,
|
||||
) {
|
||||
debug('Constructed editor page.');
|
||||
|
||||
this.route.params.subscribe(params => {
|
||||
this.pageId = params.id;
|
||||
debug('Got page ID from route:', this.pageId, params);
|
||||
@ -54,6 +56,8 @@ export class EditorPage implements OnInit {
|
||||
ngOnInit() {}
|
||||
|
||||
ionViewDidEnter() {
|
||||
debug('Ion view did enter editor page.');
|
||||
|
||||
if ( this.pageId ) {
|
||||
this.editorService.startEditing(this.pageId, this.version);
|
||||
} else if ( !this.hosted ) {
|
||||
|
@ -1,5 +1,6 @@
|
||||
import { Injectable } from '@angular/core';
|
||||
import {BehaviorSubject} from 'rxjs';
|
||||
import {debug} from '../utility';
|
||||
|
||||
@Injectable({
|
||||
providedIn: 'root'
|
||||
@ -15,6 +16,9 @@ export class NavigationService {
|
||||
}
|
||||
|
||||
requestNavigation(pageId: string) {
|
||||
debug('Requesting page navigation:', pageId);
|
||||
debug('Page navigation trace', new Error());
|
||||
|
||||
this.navigationRequest$.next(pageId);
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user