GROWI Developers Wiki

{{pagename}} - {{sitename}}

市澤樹享

kosei-n


# 20210325_再起的な移動・複製で起こる問題について

## 現状の仕様
[該当のコード](https://github.com/weseek/growi/blob/v4.2.13/src/server/service/page.js#L48)
- 親ページ単体を移動する
- isRecursibly が 指定されていた場合子ページを stream を用いて移動する
- 分けているのは親ページ処理直後にページ遷移をさせるため

## 問題点
- 余分にページが生成される可能性がある
    - `/議事録 → /20210325/議事録` のように親ページの階層を下げる場合に発生する
    - 親ページの処理の後、子ページに対する処理を行うので親ページ処理後の `/20210325/議事録` が子ページと見做され再度移動される


## 回避策

### 親子を分けずに stream 処理を行う

- pros
    - 親子が同じ処理を行うシンプルな実装になるので未来のバグも防げるだろう
- cons
    - 実行後に、処理後のページに遷移するための追加実装が必要
        - websocket で完了をブラウザに通知させる？

### 子ページの処理に親ページを含まないようにする

- pros
    - find の query を少し改修するだけなのですぐに完了する
    - 画面遷移の追加実装をする必要がない。
- cons
    - 場当たり的な改修