Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #9079 (closed enhancement: fixed)

Opened 1 year ago

Last modified 1 year ago

[PATCH] Refactored "last.is_a?(Hash) ? pop : {} " patterns to use extract_options!

Reported by: josh Assigned to: david
Priority: low Milestone: 1.x
Component: ActiveSupport Version: edge
Severity: minor Keywords: tiny
Cc:

Description

[7217] made extract_options! available to all arrays. This patch replaces instances of the "last.is_a?(Hash) ? pop : {} " to use the new syntax.

Attachments

refactored_to_use_extract_options.diff (14.2 kB) - added by josh on 07/24/07 16:39:22.

Change History

07/24/07 16:39:22 changed by josh

  • attachment refactored_to_use_extract_options.diff added.

07/24/07 16:49:00 changed by david

  • status changed from new to closed.
  • resolution set to fixed.

(In [7220]) Refactored in use of extract_options! (closes #9079) [josh]

07/24/07 16:55:40 changed by kampers

+1, applies cleanly, tests pass, looks good.

07/24/07 16:56:35 changed by kampers

haha. oh well. maybe I should preview next time.