Fosters Posted November 29, 2017 Author Share Posted November 29, 2017 5 hours ago, Maxtor said: Yes i set at least 1000 Reactions , click save, then go back-reload, and i see 0 Reactions. Seems like it doesnt save settings. could i get ACP access to take a look? Link to comment
evandixon Posted November 30, 2017 Share Posted November 30, 2017 I set the Notification Sender to a specific user, but when Trophies are awarded, the resulting notifications show as being from the user with ID 1. Medal notifications are working properly. I'd appreciate assistance in getting this working. SJ77 1 Link to comment
evandixon Posted December 2, 2017 Share Posted December 2, 2017 On 11/30/2017 at 1:32 PM, evandixon said: I set the Notification Sender to a specific user, but when Trophies are awarded, the resulting notifications show as being from the user with ID 1. Medal notifications are working properly. I'd appreciate assistance in getting this working. Fixed this myself with a manual code change, by setting ~/extensions/core/Notifications/TrophyAndMedal.php line 58 to be equal to line 69. Old value: 'author' => \IPS\Member::load( 1 ), // [Optional] The user whose photo should be displayed for this notification New value: 'author' => \IPS\Member::load( \IPS\Settings::i()->trophy_notification_sender ), // [Optional] The user whose photo should be displayed for this notification Fosters and SJ77 1 1 Link to comment
evandixon Posted December 5, 2017 Share Posted December 5, 2017 I found and fixed a race condition that results in trophies being awarded to members who do not meet the criteria. The issue is in /sources/Trophy/Trophy.php on line 193, inside the function formatFormValues. #$this->crdata = $data; $values['crdata'] = $data; $this->faicon = $values['faicon']; if ( !$this->id ) { $this->save(); } // ... return $values The problem is that `$this->crdata` is not being set before `$this->save()` is called. IPS will later use the results of this function (`$values`) to update the model, but not before an active background task can assign the trophy. The simplest fix is to simply uncomment the line of code on line 193. I have not encountered this issue after applying this fix, but because of the way race conditions work, that doesn't necessarily prove anything. Fosters 1 Link to comment
sagevil Posted December 10, 2017 Share Posted December 10, 2017 Hi i just purchased this app for my invision forum, the tab on profiles is showed but i cant find the ACP options to customize it, i mean add and change items please help. Link to comment
JTHastings Posted December 10, 2017 Share Posted December 10, 2017 It's got it's own icon in the ACP... Fosters 1 Link to comment
Fosters Posted December 10, 2017 Author Share Posted December 10, 2017 Thx @JTHastings. Yea, you have to create the trophies and medals in the ACP. Link to comment
Fosters Posted December 12, 2017 Author Share Posted December 12, 2017 (edited) On 5.12.2017 at 11:48 PM, evandixon said: I found and fixed a race condition that results in trophies being awarded to members who do not meet the criteria. The issue is in /sources/Trophy/Trophy.php on line 193, inside the function formatFormValues. #$this->crdata = $data; $values['crdata'] = $data; $this->faicon = $values['faicon']; if ( !$this->id ) { $this->save(); } // ... return $values The problem is that `$this->crdata` is not being set before `$this->save()` is called. IPS will later use the results of this function (`$values`) to update the model, but not before an active background task can assign the trophy. The simplest fix is to simply uncomment the line of code on line 193. I have not encountered this issue after applying this fix, but because of the way race conditions work, that doesn't necessarily prove anything. Hm, are you sure? This was indeed an issue in old releases, but should be fixed in the recent one. Anyway, thanks for the report, I'll review this today. Edited December 12, 2017 by Fosters Link to comment
Cyrem Posted December 13, 2017 Share Posted December 13, 2017 (edited) I deleted a category of trophies with it's contents.... Caos was unleashed. It doesn't delete the trophies from the users... and naturally, when it attempts to get info about things that don't exist... yeah. I fixed my site, but a word of warning to anyone else. Edited December 13, 2017 by Cyrem Link to comment
Fosters Posted December 13, 2017 Author Share Posted December 13, 2017 6 hours ago, Cyrem said: I deleted a category of trophies with it's contents.... Caos was unleashed. It doesn't delete the trophies from the users... and naturally, when it attempts to get info about things that don't exist... yeah. I fixed my site, but a word of warning to anyone else. Thanks, will take a look at this later today. Just to clarifym You got this screen where you choose to delete the children, correct? And the trophies were deleted, but the member associations not? Link to comment
Cyrem Posted December 13, 2017 Share Posted December 13, 2017 12 minutes ago, Fosters said: Thanks, will take a look at this later today. Just to clarifym You got this screen where you choose to delete the children, correct? And the trophies were deleted, but the member associations not? That is correct. Link to comment
Fosters Posted December 13, 2017 Author Share Posted December 13, 2017 On 2.12.2017 at 9:46 PM, evandixon said: Fixed this myself with a manual code change, by setting ~/extensions/core/Notifications/TrophyAndMedal.php line 58 to be equal to line 69. Old value: 'author' => \IPS\Member::load( 1 ), // [Optional] The user whose photo should be displayed for this notification New value: 'author' => \IPS\Member::load( \IPS\Settings::i()->trophy_notification_sender ), // [Optional] The user whose photo should be displayed for this notification thanks, fixed for the next release. I was sure that we fixed this already, but it's indeed still wrong in the recent release. Link to comment
fvasconcelos Posted December 14, 2017 Share Posted December 14, 2017 hello @Fosters, i'm thinking to change from iawards to your application. Can this display a category in the bottom post or this only shows in 1 location - postbit(user panel) ? Iawards has 2 location option to show the awards. Thanks, Link to comment
Fosters Posted December 15, 2017 Author Share Posted December 15, 2017 20 hours ago, micronx said: hello @Fosters, i'm thinking to change from iawards to your application. Can this display a category in the bottom post or this only shows in 1 location - postbit(user panel) ? Iawards has 2 location option to show the awards. Thanks, It's showing them only in the member bit and on a members profile, but we're open to suggestions:) Link to comment
Joel R Posted December 15, 2017 Share Posted December 15, 2017 On 12/14/2017 at 4:38 AM, micronx said: hello @Fosters, i'm thinking to change from iawards to your application. Can this display a category in the bottom post or this only shows in 1 location - postbit(user panel) ? Iawards has 2 location option to show the awards. Thanks, Do you mean the signature area? A major problem with the signature area is that it uses up a lot of space per post. On mobile, that signficiantly reduces screen real estate. Furthermore, many admins disable signatures to begin with since they don't really serve a compelling purpose anymore in modern forums with profiles, hovercards, avatars, and other ways of connecting with users. Link to comment
Volstate Posted December 16, 2017 Share Posted December 16, 2017 Did the Iawards conversion tool make it into this yet? I didn't see it on the change log. Link to comment
alexxis Posted December 17, 2017 Share Posted December 17, 2017 Hi @Fosters, is that possible to disable the Profile tab "Trophies and Medals" from those user's profiles, how don't have any trophies or medals? So it will be appeared only for those, who have at least one of them. Link to comment
Fosters Posted December 17, 2017 Author Share Posted December 17, 2017 59 minutes ago, alexxis said: Hi @Fosters, is that possible to disable the Profile tab "Trophies and Medals" from those user's profiles, how don't have any trophies or medals? So it will be appeared only for those, who have at least one of them. nice idea, posted to suggestion tracker https://fosters.tech/projects/item/696-hide-empty-tab/ Link to comment
Fosters Posted December 17, 2017 Author Share Posted December 17, 2017 Trophies and Medals is one of the most popular applications in the IPS Marketplace. And it's no wonder - every community wants and needs to reward users! We wanted to provide a set of links to free icons, so you can go wild while designing your own trophies and medals. Have fun and let us know what fun awards you come up with! Smashing Magazine - 55 Free High Quality Icon Sets Find Icons - Trophy IconArchive - Trophy All Free Download - Free Vector Trophy Icons Icons8 - Trophy Icons Vecteezy - Trophy Our favorite link is Smashing Magazine's 55 icon sets, which covers a variety of unusual and fun icons including Horror, Pacman, Toys, Dress Up, Royal, and Soccer. There's something in there for everyone! JTHastings and The Old Man 1 1 Link to comment
Bliss99 Posted December 19, 2017 Share Posted December 19, 2017 I don't want guests to see the trophy info if they don't have an account. How can it be removed from the guest activity feed? Link to comment
Fosters Posted December 19, 2017 Author Share Posted December 19, 2017 On 16.12.2017 at 3:33 AM, Volstate said: Did the Iawards conversion tool make it into this yet? I didn't see it on the change log. 10 hours ago, Blisslandia said: I don't want guests to see the trophy info if they don't have an account. How can it be removed from the guest activity feed? That's not possible, you'll have to disable it global Link to comment
Bliss99 Posted December 19, 2017 Share Posted December 19, 2017 3 hours ago, Fosters said: That's not possible, you'll have to disable it global So, people even if they are not members, can see details like this and there is no way to turn that off, we'd have to turn off "all activity" globally for all things? Link to comment
Fosters Posted December 20, 2017 Author Share Posted December 20, 2017 It would require custom coding Link to comment
Bliss99 Posted December 20, 2017 Share Posted December 20, 2017 13 hours ago, Fosters said: It would require custom coding Would love this feature to be in a future release - being able to apply permissions to this so that guests can't see the activity anywhere. It's important for a private community to have control over privacy that the public can not see. Link to comment
Bliss99 Posted December 20, 2017 Share Posted December 20, 2017 14 minutes ago, Adlago said: Open Applications - System and remove permissions guests for Content Discovery PP. Click on a padlock This is the way to turn off activity for guests - this will turn off all activity - which is great for now! Fosters 1 Link to comment
Recommended Posts