Add initWithCoder: method for NIB support

[ole]

DYRateView doesn't initialize correctly when it is loaded from a NIB file because the custom properties that are set in initWithFrame:fullStar:emptyStar: do not get initialized to their default values. I corrected that by adding an initWithCoder: method. Some code duplication could be removed by extracting the common initialization to a separate method. Let me know if you like this approach and I could add it, too.

[dyang]

Hi Ole,

Thank you for using DYRateView and providing the patch! I really appreciate that!

I rarely use NIBs and interface builder myself, that's probably why I haven't even tried it that way.

I'm a bit busy at this point. I'm hoping to find sometime to process your patch in the weekend.

Best regards,

Derek

Derek Yang
http://www.iappexperience.com

On Jun 29, 2012, at 4:10 PM, Ole Begemann wrote:

DYRateView doesn't initialize correctly when it is loaded from a NIB file because the custom properties that are set in initWithFrame:fullStar:emptyStar: do not get initialized to their default values. I corrected that by adding an initWithCoder: method. Some code duplication could be removed by extracting the common initialization to a separate method. Let me know if you like this approach and I could add it, too.

You can merge this Pull Request by running:

git pull https://github.com/ole/DYRateView master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

• Add constants for default full and empty images
• Add default initialization in initWithCoder: for NIB support

-- File Changes --

M DYRateView/DYRateView/DYRateView.m (18)

-- Patch Links --

https://github.com/dyang/DYRateView/pull/1.patch
https://github.com/dyang/DYRateView/pull/1.diff

---

Reply to this email directly or view it on GitHub:
#1

[dyang]

Some code duplication could be removed by extracting the common initialization to a separate method. Let me know if you like this approach and I could add it, too.

I'd very much appreciate it if you go ahead and refactor it. Thanks Ole!