Kunena / Kunena-Forum

Kunena Forum - Forum / Bulletin Board / Discussions component for Joomla - This is the 6.x/5.x main development branch. Please do not open issues regarding earlier versions of Kunena
https://www.kunena.org
GNU General Public License v3.0
1.75k stars 812 forks source link

Add emoticons to table upon installation #9724

Closed Pinkeltje closed 1 month ago

Pinkeltje commented 1 month ago

Is your feature request related to a problem? Please describe. Upon installation table kunena_smileys is empty. It can be solved by installing sample data

Describe the solution you'd like Add records upon installation. In my opinion it shouldn't be necessary to install sample data to load smileys/emoticons

https://www.kunena.org/forum/installation-issues/168526-emoticons-smileys-not-working-adding-new-emoticons-do-not-appear

rich20 commented 1 month ago

Not only smileys, but also the ranks are missing without sample data.

xillibit commented 1 month ago

Should be fixed now with my commit : https://github.com/Kunena/Kunena-Forum/commit/59b323a2d2b977904192968f02b4556b51981af4

Pinkeltje commented 1 month ago

Tested succesfully. Thx!

RakhmetKotlin commented 3 weeks ago

HI. It seems wrong to me. Emoticons have wrong code associated with them. For example (4, ':-(', '4.png', 'sad-grey.png', 1). 4.png should be :P instead of :-( and other emoticons messed up as well

Also rankdisabled and rank0 not INSERTED into database. Why ? In my Kunena download package I dont have grey emoticons png. None of 23. Where are they ?

Also all browsers seems adopted using HTML Emojis so why we still using old pngs ? We can use this instead 😭πŸ₯žπŸ³πŸ₯πŸŽ‡πŸ§¨πŸŽ„πŸŽƒπŸŽƒπŸŽƒπŸœπŸ¦ͺπŸšπŸ›πŸͺπŸŒπŸŒšπŸŒžπŸŒ β˜„οΈβ„οΈπŸ’–

rich20 commented 3 weeks ago

In my Kunena download package I dont have grey emoticons png. None of 23. Where are they ?

The grey smileys are obsolete. You have seen these in the history when you have replied to a post. The original smileys are now displayed there in K6. I will write a ticket to have the β€˜greylocation’ column removed from the database.

RakhmetKotlin commented 2 weeks ago

Thanks. But what about other issues.

Should be fixed now with my commit : 59b323a 1) rankdisabled and rank0 not INSERTED into database. Why ? 2) All emoticons have wrong code associated with them. 4.png should be :P instead of :-(

Pinkeltje commented 2 weeks ago
  1. You can always insert them if you want, but in my opinion they are useless. disabled is the same as banned and for what would you use rank0?
  2. As far as I can see, all codes show the same emoticon as the image. So I don't understand your suggested change.

    B)[img]/5/media/kunena/emoticons/1.png[/img] 8)[img]/5/media/kunena/emoticons/2.png[/img] 8-)[img]/5/media/kunena/emoticons/3.png[/img] :-([img]/5/media/kunena/emoticons/4.png[/img] :([img]/5/media/kunena/emoticons/5.png[/img] :sad:[img]/5/media/kunena/emoticons/6.png[/img] 2024-09-13 Screenshot 584

RakhmetKotlin commented 2 weeks ago

1 User can be banned on forum = rankbanned User can be banned on site = rankdisabled We can show different colored picture depending on this status.

rank0 - for new users who posted less than X number of messages

:) must show 1.png or 2.png because its a smiley face but it shows 8.png

Pinkeltje commented 2 weeks ago

Maybe better to create a new Issue/feature request as this one is closed and the subject is adding records to tables upon installation.