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	1085	September 28, 2018
Create nested entities for existing parent entity Issues	1	906	May 10, 2022
Changeset for combined relations Issues	3	816	April 10, 2018
Automatic created_at and updated_at	4	2709	November 25, 2022
Specify timestamps plugin in Root repository Issues	4	516	November 13, 2023

How can I improve this query?

Related topics