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

Ticket #10919 (closed enhancement: wontfix)

Opened 4 months ago

Last modified 4 months ago

incorrect pluralization

Reported by: srbaker Assigned to: core
Priority: normal Milestone: 2.x
Component: ActiveSupport Version: edge
Severity: normal Keywords: verified tiny
Cc:

Description

Rails improperly pluralizes the word "penis". From the New Oxford American Dictionary:

penis |ˈpēnis| noun ( pl. -nises or -nes |-nēz|) the male genital organ of higher vertebrates, carrying the duct for the transfer of sperm during copulation. In humans and most other mammals, it consists largely of erectile tissue and serves also for the elimination of urine.

As described here, there are two appropriate pluralizations of the word penis: penises or penes. The more common pluralization, penises, should be used.

While technically this is a defect, "enhancement" feels like a more appropriate word to describe this particular problem.

Attachments

penis_enhancement.diff (1.0 kB) - added by srbaker on 01/24/08 22:22:46.
Enhancement to add penis plural to the inflector.

Change History

01/24/08 22:19:30 changed by srbaker

  • component changed from ActiveRecord to ActiveSupport.

01/24/08 22:22:46 changed by srbaker

  • attachment penis_enhancement.diff added.

Enhancement to add penis plural to the inflector.

01/24/08 22:34:08 changed by ymendel

+1

01/24/08 22:34:51 changed by vinbarnes

+1

01/24/08 22:37:26 changed by rickbradley

+1 to penis enhancement. The current penis behavior is broken, without this it just won't perform properly. My expectation with rails is that I should be able to git it up and have Penis work from first pull all the way through deployment to as many servers as will come online. It may not be broken, but it's definitely badly bent.

01/24/08 22:37:51 changed by protocool

+1 - the name of the patch file surely deserves it

01/24/08 22:43:06 changed by timocratic

+1 nothing worse than not being able to handle multiple penises properly.

01/24/08 22:44:26 changed by vinbarnes

While I commend the exploratory effort that went into unearthing this grand oversight, I cannot help but comment on an additional oversight. The incorrect behavior of singularize on both 'penis' and 'penises'. This may need to be broken out into another ticket. Thoughts?

01/24/08 22:45:35 changed by nicksieger

  • keywords set to verified.

I think this made me laugh more than http://www.netfunny.com/rhf/jokes/92q4/condomman.html

01/24/08 22:53:10 changed by srbaker

@vinbarnes: The singularize behaviour is correct when using my patch. Therefore, there is no need for a bigger penis enhancement.

01/25/08 00:36:46 changed by zenspider

I'll show you a penis enhancement. :P

+1

01/25/08 01:27:11 changed by mly

  • keywords changed from verified to verified tiny.

+1

01/25/08 02:33:10 changed by ez

a verified tiny penis enhancement. now we all know. +1

01/25/08 03:37:27 changed by nzkoz

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

I appreciate that penis enhancements are the norm for most of the commenters here, but their use is definitely not widespread enough to justify fixing this.

You guys should probably investigate a plugin to handle this for you.

01/25/08 13:57:21 changed by oleganza

And the Oscar goes to srbaker. Ticket of the week, definitely.

01/25/08 15:40:25 changed by bkocik

I found a workaround:

"johnson".pluralize

=> "johnsons"

01/25/08 15:57:33 changed by josephgrossberg

+1 for being the first time I've seen a non-spam reference to "penis enhancement"

01/30/08 16:16:05 changed by kaizen

nzkoz: "You guys should probably investigate a plugin to handle this for you."

Yeah, definitly - acts_as_penis :)