add checking to length of card name

as the topic says
This check is not a good fit for the controller.  Here's why:

1. The controller is not the only thing that updates cards.  You can do it from the command line, from events, and potentially from other interfaces. None of those will now have this check.
2. The controller is not part of the developer API, and therefore this check is not something they can override.

As an important demonstration of #1, the case outlined in the story would not be caught by this code, because the issue has to do with names being set in subcards.  (You can demonstrate this by trying to add a claim with a long name).

As a rule, we're trying to move code OUT of the controller, not in.  It's really not even part of the MOVE architecture and should have as little logic as possible.  I hope to have it down under 100 lines of code soon.