Bug in VCardProvider.java: Binary avatar image doesn't get parsed

This just came up on stackoverflow and Joe is right: The image gets set to null because BINVAL is a child of PHOTO within VCARD, which gets ignored at the moment. Maybe this was so in the first drafts, back when the code was written. But the old vCard XEP and the new one, define it how it’s shown in the debug output of smack in the question on stackoverflow.

Flow

Added it to the existing defect related to vcard and issues with reading properties.

SMACK-237