search

kintone-samples / plugin-samples

55 stars 154 forks source link

SSR-1065: Gantchart: If either Start or End date is set to none/ is null, skip record and do not display #174

Closed phongnm-dev closed 3 years ago

tkmry commented 3 years ago

170 への改善案を提案してくれてありがとうございます。

内容を見ました。この更新はサブテーブルを利用した場合に意図しない非表示を起こすようです。

Thanks, for commit to #170 I checking this update. This update will cause unintentional hiding when using sub-tables. image

tkmry commented 3 years ago

上記事象の解消ですが、以下のようなコードでいかがでしょうか。 To solve the above problem, how about the following code?

            function createRecords1() {
                for (var i2 = 0; i2 < records.length; i2++) {
                    var subTable = records[i2].Table.value;
                    var displayedGanttName = false;  /* add this line */

                    for (var j = 0; j < subTable.length; j++) {
                    ...
                        var ganttRecordData = {
                            id: self.escapeHtml(records[i2]['$id'].value),
                            name: (displayedGanttName) ? '' : self.escapeHtml(records[i2][GANTT_NAME].value),  /* change this line */
                            desc:
                    ...
                        displayedGanttName = true;  /* add this line */
                        self.data.push(ganttRecordData);
                    }
                }

実際に必要な情報が表示されるようになりました。 The information the actually need is now displayed. image

commit をここで修正する方法が分からないためここに記載します。 I'm not sure how to fix commit here, so I'll list it here.

phongnm-dev commented 3 years ago

I've closed this PR, this issue will be fixed by using the newest version of Gantchart