Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Model.new에 인자로 컬럼을 집어넣기 #338

Open
shaynekang opened this issue Oct 19, 2014 · 0 comments
Open

Model.new에 인자로 컬럼을 집어넣기 #338

shaynekang opened this issue Oct 19, 2014 · 0 comments

Comments

@shaynekang
Copy link

Model.new 는 인자로 컬럼을 집어넣을 수 있습니다. 가령

message = Message.new
message.content = content
message.notice_id = notice_id

는 다음과 같이 한 줄로 줄일 수 있습니다.

message = Message.new(content: content, notice_id: notice_id)

다른 코드들도 살펴보고 Model.new에 인자로 컬럼값을 집어넣을 수 있는 부분은 리펙토링 하면 좋겠네요. ㅎㅎ
angdev added a commit that referenced this issue Oct 19, 2014
@angdev
removed SendSMS.
926e2c9 
renamed SendSMS2 to SMSSender.
Original SMSSender is Admin::Messages::SendMessageService now.

(applied #338, #339)
@angdev angdev mentioned this issue Oct 19, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@shaynekang