Assalamu alaykum,
No the hadith is sound.
The Prophet (saw) said that a man marries a woman for four reasons; her beauty, wealth, status and piety, but he should marry the one who is pious or he will be from the losers.
Then we have the hadith where the Prophet (saw) told a companion to go and look at the woman he had planned to marry, so that he can be satisfied with her looks.
These two hadiths tell us that a person should marry the one who is religious
and with whose looks he is satisfied with.
If a man sought a woman for her piety, then after he saw her, he didn’t like her looks, he’d be rejecting a religious woman. It would be wrong to show interest in a religious woman then reject her due to her looks. But if he has already seen her first and is satisfied with her appearance, then he enquires about her piety and is not satisfied with it, he will be rejecting her because of her lack of religiousness, and that is the right reason to reject a woman for. Hence, this will prevent a man rejecting a religious woman due to her looks.
If it is not, then how can you reject a woman based on her lack of beauty even though her deen satisfied you?
If a person is married for their religiousness alone, and there is a lack of physical affection, it can lead to him looking at other women and can lead to fitnah. However, that does not mean that a person should marry for beauty alone either. Rather it means that a person should marry the one who is religious and with whose appearance they are pleased with. InshaAllah, I hope I explained it alright?
:sl: