いつもお世話になっております。

現在、DomaをSpringと一緒に使ってまして、コンテナ管理されているPOJOをDaoメソッドに渡しても、SQL組み立て時にパラメータで受け取ったPOJOから値が取れない状況です。

org.seasar.doma.internal.expr.ExpressionEvaluator#getFieldValueでtargetから値が取れてません。

POJOをコンテナ管理にしているので、CGLIBでエンハンスされ、
クラスが Hoge$EnhancerByCGLIB$$xxxxxx のようになってますが、
Fieldは元のクラス(Hoge)の情報を使っているからだと思っています。

newしたHogeを渡すと値が取れます。

良い解決策があれば教えてください。

DaoGeneratorをカスタマイズ出来るのであれば、エンハンス元のクラスのインスタンスを生成してディープコピーしようかなと思ってます。

POJO

@Named
@Scope(proxyMode = ScopedProxyMode.TARGET_CLASS, value = "request")
public class Hoge {
    private String fuga;
    // getter, setter省略
}

Dao

@Dao
@DaoConfig  // Spring連携用の自作アノテーション
public interface ChargeRepository {
    @Insert(sqlFile = true)
    int create(String p0, Hoge hoge);
}

SQLファイル

insert into table01
select *
     , /*hoge.fuga*/'a'     -- hoge.fugaがnullになる
  from table02
 where col1 = /*p0*/'a'

Hi
It would be nice to have a logo icon or image of doma so that we can use it for introducing doma.

お世話になります。

@infixについて質問となりますが、
Doma2では%abc%のように生成されるのでしょうか？
それとも%abc%のように生成されるのでしょうか？

よろしくお願いいたします。

インターフェースかつメソッドに定義されたアノテーションは継承されないため、Daoに適用したインターセプタから@selectなどのアノテーションの情報を取得しようとすると、実装クラスから親を辿って@daoが定義されたインターフェースを探し、そのインターフェース定義にある同名かつ互換性のある型の引数を持つメソッドを探し、そのアノテーションを取得するというステップが必要になります。
特にオーバーロードされたメソッドがいくつもあると、正しく「同名かつ互換性のある型の引数を持つメソッドを探す」のが複雑で、またそれなりに遅くなりそうです。

そのため、Daoのインターフェースに定義された@selectなどのアノテーションを、自動生成された実装クラスにも引き継いで定義することを検討していただけないでしょうか。

https://twitter.com/tkatochin/status/461681621452595200

代わりにセーブポイント作成のログが出力されてしまっている。

Hi,

It seems that Doma creates wrong insert sql for an entity only with auto generated ID column like this:

insert into ID_ONLY) values)

It might be better to generate following query:

insert into ID_ONLY() values ()

I'm glad if you kindly share your opinion about this issue.
I will send a pull request.

お世話になります。

「アプリで発行された更新系SQLを全てDBに保存する」という要件があるのですが、
Loggerで取れる上記の情報をアプリ側で取得する方法はありますか？

以前は、Seasar2(Teeda + S2Dao)を使っていたのですが、
その時はインターセプタ内で、SqlLogRegistryLocator.getInstance().getLast()で取得して記録していました。

今使っているDIコンテナはSpringです。

よろしくお願いします。

はじめまして。

以下の要望が現在の仕組みで実現可能かお聞きしたいです。
もしくは、今後の機能拡張案としていただけると幸いです。

要望：SQLの先頭に自動でコメントを付与したいです。

理由：RDB側でリストアップされた遅いSQLの先頭にあるコメントを見て、
　　　アプリ側（該当SQLの実行元）と紐付けたいためです。
　　　リストアップされたSQLは後ろが切れたりWhere句が変わるため、該当SQLを探すのが困難です。

備考：コメントはRDB側とアプリ側で紐付け出来れば何でも構いません。
　　　SQLファイルパスが付与できれば分かりやすいです。
　　　主にSQL数が多く、似たSQLが多いアプリケーションで需要があると考えます。
　　　パフォーマンス問題が発生した際、該当SQLの実行元を調査する際に有用です。
　　　手動でコメントを付与するのは、量の多さや記法の不統一の恐れから避けたいです。
　　　Domaで実現せず、アプリとDBを透過的に監視するソリューションを使う案もあるとは思います。

自身で変更する場合の案：
　　　ソースを拝見した限りでは、prepareSqlメソッドの付近で処理を新規追加するのが楽だと思いました。
　　　見栄えを考えると、DaoImplクラスの__query.set*** の並びにあると分かりやすいと思いました。

@OriginalStatesがあるEntityの値を変えながら複数回UPDATEすると、2回目以降のUPDATEで変更した項目が正しく反映されないことがあります。

次のようなテストコードで検証できるのですが、2回目以降のUPDATEの際に最初のSELECT時の値と同じ値に変更すると、正しく変更が検知されていないようです。

HogeEntity entity = new HogeEntity();
entity.setId(1);
entity.setValue("A");
dao.insert(entity);

entity = dao.selectById(1);

entity.setValue("X"); //Xに更新
dao.update(entity);

entity.setValue("A"); //Aに戻す
dao.update(entity);

HogeEntity actual = dao.selectById(1);

assertEquals("A", actual.getValue());

PostUpdateの処理の中で毎回@OriginalStatesに更新時の値を反映するようにすれば解決しそうですが、この挙動は意図した仕様なのでしょうか？

自動生成されたDaoの実装クラスに引数なしのコンストラクタが無いため、CDIによってインスタンス化される際にエラーになります。

下記リンク先が参考になりますが、OpenWebBeansによるCDI実装でインターセプタが適用される場合、まず引数ありのコンストラクタを呼び出してインスタンスを作成した後に、プロキシークラスを生成して引数なしのコンストラクタを呼び出すので、そこでエラーになります。

http://stackoverflow.com/questions/15637391/should-cdi-bean-with-interceptors-have-a-default-constructor

CDIの仕様も確認してみたところ、Managede Beanとしては引数なしのコンストラクタが必須ではないのですが、プロキシー化が必要な場合には引数なしのコンストラクタが必須となっています。

WeldによるCDI実装では問題なさそうなのですが、OpenWebBeansによるCDI実装では仕様としてエラーになっているようです。

自動生成されたDaoの実装クラスに引数なしのコンストラクタを追加するようにDomaの改善を検討してみていただけないでしょうか。

https://github.com/seasarorg/doma/blob/1.36.0/doma/src/main/java/org/seasar/doma/jdbc/tx/LocalTransactionContext.java#L78

subListへの引数がまちがっている。

単純なものですし、修正となると1.38.0リリースになると思いましたのでプルリクエストではなくIssue登録にしました。
ご確認ください。

• https://github.com/seasarorg/doma/blob/1.37.0/doma/src/main/java/org/seasar/doma/internal/Artifact.java#L29

このリポジトリ、つまり seasarorg/doma はバージョン1専用とします。
バージョン2は domaframework/doma で開発します。