# リサーチ情報、事前設定項目定数が残っている箇所

## 項目追加の実装漏れの可能性がある箇所のみまとめ

### CSV エクスポート画面 検索項目
項目が足りないかもしれないです。

### OP架電画面->印刷ボタン
項目が足りないかもしれないです。

## 1. 現状のinterface で型表記が固定されているのに何故動作しているのかについての調査結果

以下何故動作しているのか気になったので調べた結果の報告です。主に以下の2種類のことが起こっており現状動作しているのだと思っています。

### ① vue ファイルでデータを参照している Storeがそもそも異なっている
こちら中村さんの修正で意図的にやっていただいたと思うのですが、.vue ファイルでデータを参照している Storeがそもそも異なっているので動作している箇所ありました。
現状テンプレートでは 事前設定項目、リサーチ情報は ItemCustomStore.ts で参照していると思います。
この場合、MasterStore.ts や他のstore で今までの記述が残り、そこに、リサーチ情報と事前設定項目の直書き　の記述(以前の仕様)が残っている様な状態に現状のブランチではなっているのではと考えています。

### ② 現在の利用で、新しく追加された項目を取得する記述がない(response 用のinterfaceなど)

response 用のinterface に直書きでかかれていてそれを利用しているのに動いている理由について、
axios通信をした結果(response.data等)で今回追加していただいた箇所(research11とか)をkey名を指定して、取得していないから不具合が発生していないのだと思います。なので今後取り出そうとする記述を書く際に困るかなと思います。

### ③ interfaceに定義が足りない箇所はイベント発火時に連想配列がついかされている(checkボックス、フォーム等追加)
interfaceに定義が足りない箇所はイベント発火時に連想配列が追加されているようです。
例)
事前情報1 を編集した場合は
事前に用意してあるadvancedInformation1というkeyにvalueがバインドされる。
事前情報11 を編集した場合は
advancedInformation11 の連想配列が新たに作成され値がバインドされるため動作している。初期表示だと、事前設定項目は8までだけあります。


## 2. 直書きされている箇所

以前の仕様のままのソースと考え 「research3」で検索した結果の箇所全てのファイルを下に列挙しました。

### 140 件の結果 - 22 ファイル

### `AdminCallingDetail.vue`

リサーチ情報をMasterstore に入れる記述を直書きをしているがMasterStoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。

### `BaseSearch.vue`

　自分の変更で削除済みな気がします。
[https://github.com/scene-live/list-navigator/pull/1799](https://github.com/scene-live/list-navigator/pull/1799)

### `OpPrint.vue`

リサーチ情報をMasterstore に入れる記述を直書きをしているがMasterStoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。

### ==`UpdateListRequest.ts`==

import ないので、使われていない?使用箇所の記述が見つかりませんでした。

### `ListDeleteRequest.ts`

store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループいるため動いていると思います。削除しても問題なければ削除でいいかもしれないです

### `ListSearchRequest.ts`

store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ削除でいいかもしれないです

### `AdminCallingResponse.ts`
research11 以降、advancedInformation９以降使っているかもしれないですが、直接key名を指定して使っていないため動いていると思います。

---
OperatorSearchStore.ts updateList()
でinterface に定義されていない keyがParam としてpost されているような気がします。

<確認方法>
管理者架電　-> 情報をセットして次へを押下し確認しました。

[OperatorSearchStore.ts 一部]
    @Action
    public async updateList(args: {
        operatorId?: number,
        sipId?: string;
        number?: string;
        recordName?: string;
        type?: string;
        updateType?: number;
        isNotHistoryRegist?: number;
        actualCallFlag?: number;
    }): Promise<boolean> {
        
        interface Param {
            updateParams: ListDetailData;
            listId: number;
            operatorId: number;
            isNotHistoryRegist: number;
            sipUser?: string;
            dstNumber?: string;
            recFileName?: string;
            type?: string;
            updateType?: number;
            actualCallFlag?: number;
        }
        let param: Param = {
            updateParams: this._listDetailData,
            listId: Number(this._listDetailData.listId),
            operatorId: (args.operatorId) ? args.operatorId : 1,
            isNotHistoryRegist: (args.isNotHistoryRegist) ? args.isNotHistoryRegist : 0,
            recFileName: args.recordName,
            type: args.type,
            updateType: args.updateType,
            actualCallFlag: args.actualCallFlag,
        };

(以下省略)

型安全怪しいので修正必要かもしれません。

### `MasterDataResponse.ts`

research11 以降、advancedInformation９以降key名を指定して使っていないため動いていると思います。
追加した項目を今の記述のまま利用しようとした場合に企業によってはキーが存在せずエラーになると思います。

- OperatorSearchStore.ts （１例です）
現状は追加した項目のkey名の指定を使用していないためエラーになっていないかと。

Store側 [OperatorSearchStore.ts]
    @Action
    public async initMaster(targetMasterKeys?: string[]): Promise<boolean | BaseError> {
        if (!targetMasterKeys) {
            targetMasterKeys = ['callStatus'];
        }

        try {
            // ここ戻り値型安全をresearch10まで行っている
            const result = await post<MasterDataResponse, {}>(GET_MASTER(targetMasterKeys));
        以下略

--

インターフェイス側：　[MasterDataResponse.ts]

 export interface MasterDataResponse extends BaseResponse {
     data: {
         displayMasterDataList: any;
         masterDataList: MasterDataList;
     };
 }
 
// research11でkey 名を指定して取得できない状況
export interface DisplayMasterDataList  {
    itemTitle: string[] | null;
    pref: string[] | null;
    listGroup: string[] | null;
    research1: string[] | null;
    research2: string[] | null;
    research3: string[] | null;
    research4: string[] | null;
    research5: string[] | null;
    research6: string[] | null;
    research7: string[] | null;
    research8: string[] | null;
    research9: string[] | null;
    research10: string[] | null;
    week: Week[] | null;
    banStatus: BanStatus[] | null;
    callStatus: CallStatus[] | null;
    predictStatus: CallStatus[] | null;
}

### `OpCallResponse.ts`

research11 以降、advancedInformation９以降key名を指定して使っていないため動いていると思います。
追加した項目を今の記述のまま利用しようとした場合に企業によってはキーが存在せずエラーになると思います。
(MasterDataResponset.ts と同じ)

### `CompanyAllocationStore.ts`

store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです

### `CompanyReleaseStore.ts`
store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです

### `CsvExportStore.ts`

CSVエクスポート 検索項目足りない気がします。自分の環境だと画像のようになりました。

### `ListCleaningStore.ts`

store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `ListCreateCallingStore.ts`
store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `ListDeleteStore.ts`
store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `ListOpToOpStore.ts`
store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `MasterDataStore.ts`

### OP画面->印刷ボタンで表示項目が足りない気がします。

### `OperatorAllocationStore.ts`

store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `OperatorReleaseStore.ts`
store に入れる記述を直書きをしているがstoreの値をテンプレートで使用せず、itemCustomStoreを見てループしているため動いていると思います。削除しても問題なければ直書きの箇所削除でいいかもしれないです。

### `OperatorCallStore.ts`
resetListData() メソッドで初期化できていない気がします。
使われてはいそうですが、事前情報とリサーチ情報に関しては、itemCustomStoreを見てループしているため動いていると思います。

### `OperatorSearchStore.ts`
resetListData() メソッドで初期化できていない気がします。
使われてはいそうですが、事前情報とリサーチ情報に関しては、itemCustomStoreを見てループしているため動いていると思います。

### `PredictiveListStore.ts`

RESET_SEARCH_FORM() メソッドで初期化できていない気がします。

---