Showing 1,608 of 1,614 total issues
Stats::Actions#done_last30days_data has the variable name 'i' Open
created_count_array = Array.new(30) { |i| @actions_created_last30days.size / 30.0 }
done_count_array = Array.new(30) { |i| @actions_done_last30days.size / 30.0 }
# TODO: make the strftime i18n proof
time_labels = Array.new(30) { |i| I18n.l(Time.zone.now - i.days, :format => :stats) }- Read upRead up
- Exclude checks
An Uncommunicative Variable Name is a variable name that doesn't communicate its intent well enough.
Poor names make it hard for the reader to build a mental picture of what's going on in the code. They can also be mis-interpreted; and they hurt the flow of reading, because the reader must slow down to interpret the names.
Stats::Actions#put_events_into_month_buckets has the variable name 'r' Open
convert_to_array(records.select { |x| x.send(date_method_on_todo) }, array_size) { |r| [difference_in_months(@today, r.send(date_method_on_todo))] }- Read upRead up
- Exclude checks
An Uncommunicative Variable Name is a variable name that doesn't communicate its intent well enough.
Poor names make it hard for the reader to build a mental picture of what's going on in the code. They can also be mis-interpreted; and they hurt the flow of reading, because the reader must slow down to interpret the names.
Stats::Actions#time_of_day_all_data has the variable name 'i' Open
@actions_creation_hour_array = Array.new(24) { |i| 0 }
@actions_creation_hour.each { |r| @actions_creation_hour_array[r.created_at.hour] += 1 }
# convert to hash to be able to fill in non-existing days
@actions_completion_hour_array = Array.new(24) { |i| 0 }- Read upRead up
- Exclude checks
An Uncommunicative Variable Name is a variable name that doesn't communicate its intent well enough.
Poor names make it hard for the reader to build a mental picture of what's going on in the code. They can also be mis-interpreted; and they hurt the flow of reading, because the reader must slow down to interpret the names.
Stats::Actions#visible_running_time_data has the variable name 'i' Open
time_labels = Array.new(@count) { |i| "#{i}-#{i + 1}" }- Read upRead up
- Exclude checks
An Uncommunicative Variable Name is a variable name that doesn't communicate its intent well enough.
Poor names make it hard for the reader to build a mental picture of what's going on in the code. They can also be mis-interpreted; and they hurt the flow of reading, because the reader must slow down to interpret the names.
RecurringTodos::AbstractRecurrencePattern#find_xth_day_of_month has the parameter name 'x' Open
def find_xth_day_of_month(x, weekday, month, year)- Read upRead up
- Exclude checks
An Uncommunicative Parameter Name is a parameter name that doesn't communicate its intent well enough.
Poor names make it hard for the reader to build a mental picture of what's going on in the code. They can also be mis-interpreted; and they hurt the flow of reading, because the reader must slow down to interpret the names.
TODO found Open
# TODO: make the strftime i18n proof- Exclude checks
TODO found Open
# TODO: if there is still no match, start will be set to nil. if we ever- Exclude checks
TODO found Open
// TODO: clean these up after the migration to Bootstrap is complete- Exclude checks
TODO found Open
// TODO: wrong method name. refactor to remove this method- Exclude checks
FIXME found Open
# FIXME: These should also be used.- Exclude checks
TODO found Open
# TODO: parameterize limit- Exclude checks
TODO found Open
// TODO: remove when dropping the .bootstrap namespace- Exclude checks
TODO found Open
/* TODO: blocking of dropdown */- Exclude checks
TODO found Open
# TODO: test this for eager and not eager loading!!!- Exclude checks
TODO found Open
# TODO: remove next line if 'project' supports group_view_by- Exclude checks
HACK found Open
# HACK: This is something of a hack to use the correct operator for Pg- Exclude checks
TODO found Open
# TODO: refactor to remove outer hash from includes param- Exclude checks
TODO found Open
/* TODO: make this generic for all pages with lists */- Exclude checks
TODO found Open
# TODO: I think this will work, but can't figure out how to test it- Exclude checks
TODO found Open
# New todo - TODO: Filter on current project in project view- Exclude checks