Skip to content
This repository has been archived by the owner on Jun 16, 2020. It is now read-only.

Symbols fixup #1

Merged
merged 1 commit into from Oct 13, 2011
Merged

Symbols fixup #1

merged 1 commit into from Oct 13, 2011

Conversation

mat813
Copy link

@mat813 mat813 commented Oct 13, 2011

It makes bitmasks easier to use in Rails applications because you don't
have to wonder if you did cast what you got from your web forms.

it keeps hitting me and I end up with things like that :

>> c.code
[
    [0] :power,
    [1] :visuprod,
    [2] :no_tva
]
>> c.code << "power"
[
    [0] :power,
    [1] :visuprod,
    [2] :no_tva,
    [3] "power"
]

and it kinda confuse my app from time to time :-)

@mat813
Make sure our array only contain symbols.
088a9fe 
It makes bitmasks easier to use in Rails applications because you don't
have to wonder if you did cast what you got from your web forms.
joelmoss added a commit that referenced this pull request Oct 13, 2011
@joelmoss
Merge pull request #1 from Absolight/symbols_fixup
b4d1722 
Symbols fixup to ensure that values are always returned as symbols regardless of how they were cast.
@joelmoss joelmoss merged commit b4d1722 into joelmoss:master Oct 13, 2011
@joelmoss
Copy link
Owner

Thx buddy!

@spemmons spemmons mentioned this pull request Mar 21, 2012
Merged
spemmons added a commit that referenced this pull request Apr 2, 2012
@spemmons
Merge pull request #1 from numerex/master
6eb7bff 
move activity to a public repository
marcelotedeschi pushed a commit to marcelotedeschi/bitmask_attributes that referenced this pull request Nov 9, 2018
@alpaca-tc @marcelotedeschi
Merge pull request joelmoss#1 from pixiv/evaluate_with_method_location
b15e2ce 
Evaluate with method location
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mat813 @joelmoss