flowable / flowable/flowable-engine

Forms REST API missing Form Instance Update

オープン
#2,101 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
9.5k
フォーク
2.9k
平均マージ
7時間 8分
マージ済み PR(30日)
2

説明

**Describe the bug**
The POST body to `/form-api/form/form-instances` accepts a formInstanceId, but it is not used. The Forms Java API provides provides both `createFormInstance` and `saveFormInstance` methods, but the REST API only uses the `createFormInstance` methods.

**Expected behavior**
Standard REST practice would be to have PUT endpoint to update an existing FormInstance, but overloading the POST to save instead of creating when a formInstanceId is possible.

**Additional context**
The Task app uses the `saveFormInstance` when updating a task with a form, implying that updating existing is the expected behavior.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

REST エンドポイント /form-api/form/form-instances から開始し、その処理を Forms Java API のメソッド createFormInstance および saveFormInstance と比較します。formInstanceId が現在どのように処理されているかを追跡し、既存のインスタンスでは save の動作が使用され、新しいインスタンスでは create の動作が維持されることを確認します。これには、想定される PUT のサポートも含まれます。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
api, backend
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。