Project

General

Profile

Actions

Defect #4056

closed

Rename named_scope :public for Project to something else

Added by Dmitri Kouikoff over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Projects
Target version:
Start date:
2009-10-19
Due date:
% Done:

100%

Estimated time:
Resolution:
Fixed
Affected version:

Description

As the word public is reserved in ruby, the attempt to mock/stub the model Project fails, since the line that introduces a new public method fails due to the fact that word 'public' is redefined by the named_scope.

I suspect you may meet the case while setting up the shoulda.

Actions #1

Updated by Eric Davis over 14 years ago

  • Category set to Projects
  • Status changed from New to 7

Agreed. I ran into this when I would mock Project.public in my tests. What do you propose renaming it to? Project.public_projects? Project.all_public?

Actions #2

Updated by Dmitri Kouikoff over 14 years ago

I think :all_public is pretty good. As a variant I was thinking about :public_state. :-)

Actions #3

Updated by Eric Davis over 14 years ago

  • Status changed from 7 to Closed
  • Target version set to 0.9.0
  • % Done changed from 0 to 100
  • Resolution set to Fixed

Project.all_public it is. Thanks for the suggestion. r2941

Actions

Also available in: Atom PDF