fix formating bug

[stanbar]

getQuantityString require twice if you want formating From docs

When using the getQuantityString() method, you need to pass the count twice if your string includes string formatting with a number. For example, for the string %d songs found, the first count parameter selects the appropriate plural string and the second count parameter is inserted into the %d placeholder. If your plural strings do not include string formatting, you don't need to pass the third parameter to getQuantityString.

and you forgot to set the visibility if the hashtag.count is not null

[hanggrian]

and you forgot to set the visibility if the hashtag.count is not null

This is an interesting topic. When a count is not null and 0, should count view also be gone?

[stanbar]

Yes, I would like to show 0 count as well

[hanggrian]

hashtag.count null checks combined, and merged. Thanks for your contribution!

[stanbar]

Hello my friend. The same thing happens in MentionArrayAdapter You set the text

            if (!mention.displayname.isNullOrEmpty()) {
                holder.displaynameView.text = mention.displayname
            }

but doesn't change visibility, which is always

        <TextView
            android:id="@+id/socialview_mention_displayname"
            style="?android:attr/dropDownItemStyle"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:ellipsize="marquee"
            android:singleLine="true"
            android:textAppearance="?textAppearanceLargePopupMenu"
            android:visibility="gone" />

gone

[hanggrian]

I will investigate this, but I warn you, it could take some time. This library is undergoing downgrade to Java.

[stanbar]

Why is that ? :O

[hanggrian]

I began to question that myself. But I'm supposed it's for lower dependencies since not everyone uses Kotlin. And this is a fairly simple project, Kotlin might have been an overkill.