fix: fix type error

dream-num · Jun 18, 2024 · 068245e · 068245e
1 parent 721f501
commit 068245e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/packages/docs/src/commands/commands/delete.command.ts b/packages/docs/src/commands/commands/delete.command.ts
@@ -55,7 +55,7 @@ export const DeleteLeftCommand: ICommand = {
  const docSkeletonManagerService = getCommandSkeleton(accessor, unitId);
  const activeRange = textSelectionManagerService.getActiveRange();
  const ranges = textSelectionManagerService.getSelections();
- const skeleton = docSkeletonManagerService?.getSkeleton()?.skeleton;
+ const skeleton = docSkeletonManagerService?.getSkeleton();
  if (activeRange == null || skeleton == null || ranges == null) {
  return false;
  }
@@ -202,7 +202,7 @@ export const DeleteRightCommand: ICommand = {
 
  const activeRange = textSelectionManagerService.getActiveRange();
  const ranges = textSelectionManagerService.getSelections();
- const skeleton = docSkeletonManagerService?.getSkeleton()?.skeleton;
+ const skeleton = docSkeletonManagerService?.getSkeleton();
  if (activeRange == null || skeleton == null || ranges == null) {
  return false;
  }

diff --git a/packages/docs/src/controllers/move-cursor.controller.ts b/packages/docs/src/controllers/move-cursor.controller.ts
@@ -99,7 +99,7 @@ export class MoveCursorController extends Disposable {
  }
 
  const skeleton = this._renderManagerService.getRenderById(docDataModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
  const docObject = this._getDocObject();
 
  if (activeRange == null || skeleton == null || docObject == null) {
@@ -203,7 +203,7 @@ export class MoveCursorController extends Disposable {
  }
 
  const skeleton = this._renderManagerService.getRenderById(docDataModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
  const docObject = this._getDocObject();
 
  if (activeRange == null || skeleton == null || docObject == null || allRanges == null) {

diff --git a/packages/docs/src/controllers/normal-input.controller.ts b/packages/docs/src/controllers/normal-input.controller.ts
@@ -65,7 +65,7 @@ export class NormalInputController extends Disposable {
  const e = event as InputEvent;
 
  const skeleton = this._renderManagerService.getRenderById(documentModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
 
  if (e.data == null || skeleton == null) {
  return;

diff --git a/packages/engine-render/src/render-manager/render-manager.service.ts b/packages/engine-render/src/render-manager/render-manager.service.ts
@@ -50,7 +50,7 @@ export interface IRenderManagerService extends IDisposable {
  /** @deprecated */
  createRender$: Observable<string>;
  /** @deprecated this design is very very weird! Remove it. */
- create(unitId: Nullable<string>): void;
+ create(unitId: string): void;
 
  /** @deprecated There will be multi units to render at the same time, so there is no *current*. */
  getCurrent(): Nullable<IRender>;
@@ -127,7 +127,7 @@ export class RenderManagerService extends Disposable implements IRenderManagerSe
  return Array.from(this._renderControllers.get(type) ?? []);
  }
 
- create(unitId: Nullable<string>) {
+ create(unitId: string) {
  this._createRender$.next(unitId);
  }