actually disregard that.  I guess this is fine.  the fact that you added a call to `.trim()` and checked the result to make sure it wasn't an empty attribute made me think that this was all new code.  Was there a reason to add the call to `trim()` ?