It's in poor taste, to say the least. The last Protestant (Baptist) church I went to would sell branded merchandise (much like what you posted), and also have guest speakers who would advertise their own products during the "sermon". One Sunday service consisted of a football player bragging about his career for an hour and then advertising his books for the last 20 minutes.
During my conversion to Catholicism, I was blown away by all the free things I was given by the Church.
>catechism of the catholic church
>missal
>books/pamphlets about advent, lent, etc.
>DVDs explaining the Faith
>rosary beads w/ how-to pamphlet
And countless other things. The last time I went to Mass, they had all sorts of free material laid out, and I brought a really nice Catholic calendar home. I'm not saying every church should be that generous, however I'd argue that turning your church into a marketplace is worthy of condemnation.