Issues/247

Findbugs reporting now integrated and build failing on high priority issues.  All of the high priority issues that have been reported have been removed, and a couple of additional tests added.  Lots of minor style/convention issues (equals method, string constants, literal booleans in conditionals and other bits and pieces).  It probably went further than I wanted it to.
```java
for (NodeMembership ns : cur) {
                if (!actorJid.toBareJID().equals(ns.getUser().toBareJID()) &&
                    !isOwnerModerator() &&
                    !ns.getSubscription().equals(Subscriptions.subscribed)) {
                    continue;
                }
                Element subscription = subscriptions.addElement(XMLConstants.SUBSCRIPTION_ELEM);
                subscription.addAttribute(XMLConstants.NODE_ATTR, ns.getNodeId())
                        .addAttribute(XMLConstants.SUBSCRIPTION_ELEM, ns.getSubscription().toString())
                        .addAttribute(XMLConstants.JID_ATTR, ns.getUser().toBareJID());
                if (null != ns.getInvitedBy()) {
                    subscription.addAttribute(XMLConstants.INVITED_BY_ELEM, ns.getInvitedBy().toBareJID());
                }

            }
```

...would give us the correct logic