How can I improve this query?

elithecho · April 18, 2022, 11:37am

I have a small repo for trying out ROM and using multi_insert. I think it could be improved but am wondering how can I do this without a separate query?

github.com

elithecho/rom-test/blob/main/lib/app/repos/user_repo.rb#L21-L39

      
        
            def create_with_tasks(user)
              users.transaction do
                task_attrs = user[:tasks]
                user = users.changeset(:create, user).map(:add_timestamps).commit
            
            
    # It does not set the timestamps
                # https://www.rubydoc.info/github/rom-rb/rom-sql/ROM/SQL/Relation
                # https://www.rubydoc.info/github/rom-rb/rom-sql/ROM/SQL/Relation#multi_insert-instance_method
                new_tasks = tasks.multi_insert(task_attrs.map do |task|
                  task.merge!(
                    user_id: user.id,
                    created_at: Time.now,
                    updated_at: Time.now
                  )
                end)
              end
            
            
  get_by_id(user.id).combine(:tasks).one
            end

Topic		Replies	Views
Create multiple entities Usage Stories	1	1074	September 28, 2018
Changeset for combined relations Issues	3	763	April 10, 2018
Create nested entities for existing parent entity Issues	1	890	May 10, 2022
How do you set timestamps in version 4.0? Site Feedback	7	2247	April 2, 2018
Rom-sql upsert multiple entities by default Usage Stories	8	2748	June 28, 2018

How can I improve this query?

Related topics