Redmine

      tags << options_for_select(values.map {|text, value| [text, "value:#{value}"]}, import.mapping[field] || options[:default_value])

  def allowed_target_users

    users = []

    if project

      users = project.members.active.preload(:user)

      users = users.map(&:user).select{ |u| u.allowed_to?(:log_time, project) }

    end

    users << User.current if User.current.logged? && !users.include?(User.current)

    users

  end

  def user_value

    if mapping['user_id'].to_s =~ /\Avalue:(\d+)\z/

      $1.to_i

    end

  end

    object.author = user

    user_id = nil

    if User.current.allowed_to?(:log_time_for_other_users, project)

      user_id = user_value || row_value(row, 'user_id')

    else

      user_id = user.id

    end

      :user_id     => user_id,

<% if User.current.allowed_to?(:log_time_for_other_users, @import.project) %>

  <p>

    <label for="import_mapping_user_id"><%= l(:field_user) %></label>

    <%= mapping_select_tag @import, 'user_id', :required => true,

          :values => @import.allowed_target_users.map {|u| [u.name, u.id]}, :default_value => "value:#{User.current.id}" %>

  </p>

<% end %>

    <th><%= t(:field_user) %></th>

    <td><%= link_to_user time_entry.user %></td>

row;issue_id;date;hours;comment;activity;overtime;user_id

1;;2020-01-01;1;Some Design;Design;yes;2

2;;2020-01-02;2;Some Development;Development;yes;2

3;1;2020-01-03;3;Some QA;QA;no;3

4;2;2020-01-04;4;Some Inactivity;Inactive Activity;no;2

  def test_get_mapping_time_entry

    Role.find(1).add_permission! :log_time_for_other_users

    import = generate_time_entry_import

    import.settings = {'separator' => ";", 'wrapper' => '"', 'encoding' => "ISO-8859-1"}

    import.save!

    get :mapping, :params => {

        :id => import.to_param

      }

    assert_response :success

    # 'user_id' field should be available because User#2 has both

    # 'import_time_entries' and 'log_time_for_other_users' permissions

    assert_select 'select[name=?]', 'import_settings[mapping][user_id]' do

      # Current user should be the default value

      assert_select 'option[value="value:2"][selected]', :text => User.find(2).name

      assert_select 'option[value="value:3"]', :text => User.find(3).name

    end

  end

  def test_get_mapping_time_entry_for_user_without_log_time_for_other_users_permission

    import = generate_time_entry_import

    import.settings = {'separator' => ";", 'wrapper' => '"', 'encoding' => "ISO-8859-1"}

    import.save!

    get :mapping, :params => {

        :id => import.to_param

      }

    assert_response :success

    assert_select 'select[name=?]', 'import_settings[mapping][user_id]', 0

  end

  def generate_time_entry_import(fixture_name='import_time_entries.csv')

    import = TimeEntryImport.new

    import.user_id = 2

    import.file = uploaded_test_file(fixture_name, 'text/csv')

    import.save!

    import

  end

    User.current = nil

  def test_maps_user_id_for_user_with_permissions

    User.current = User.find(1)

    import = generate_import_with_mapping

    first, second, third, fourth = new_records(TimeEntry, 4) { import.run }

    assert_equal 2, first.user_id

    assert_equal 2, second.user_id

    assert_equal 3, third.user_id

    assert_equal 2, fourth.user_id

  end

  def test_maps_user_to_column_value

    User.current = User.find(1)

    import = generate_import_with_mapping

    import.mapping.merge!('user_id' => 'value:1')

    import.save!

    first, second, third, fourth = new_records(TimeEntry, 4) { import.run }

    assert_equal 1, first.user_id

    assert_equal 1, second.user_id

    assert_equal 1, third.user_id

    assert_equal 1, fourth.user_id

  end

  def test_maps_user_id_for_user_without_permissions

    # User 2 doesn't have log_time_for_other_users permission

    User.current = User.find(2)

    import = generate_import_with_mapping

    first, second, third, fourth = new_records(TimeEntry, 4) { import.run }

    assert_equal 2, first.user_id

    assert_equal 2, second.user_id

    # user_id value from CSV should be ignored

    assert_equal 2, third.user_id

    assert_equal 2, fourth.user_id

  end

        'comments'   => '4',

        'user_id'    => '7'