Adriano Faria Posted July 25, 2018 Author Posted July 25, 2018 (edited) 9 hours ago, Brainy S. said: I would be extremely curious to know if you are able to replicate this problem based on the steps I provided, or not. I can confirm the hook working even with the app setting disabled, that is fixed, but I got no error when I change the author. Tested on my dev board (4.3.5), author is changed, no error logged. Tested on my online board (4.3.4), no error logged: Let me know if I did something different than your post. If I did, please, be clear and straight. Edited July 25, 2018 by Adriano Faria
newbie LAC Posted July 25, 2018 Posted July 25, 2018 - Log in as member and create new topic - Log in as admin (note this admin shouldn't have linked accounts or turn of the setting Allowed Groups to POST AS another member) and edit that topic. Now topic author is Guest BTW the above setting is not displayed correctly on ACP $form->add( new \IPS\Helpers\Form\Select( 'la_groups_postas', \IPS\Settings::i()->la_groups_protected == '*' ? "*" : explode( ',', \IPS\Settings::i()->la_groups_postas )
Adriano Faria Posted July 25, 2018 Author Posted July 25, 2018 3 minutes ago, newbie LAC said: - Log in as admin (note this admin shouldn't have linked accounts or turn of the setting Allowed Groups to POST AS another member) and edit that topic. Yes, it wasn't checking the setting. Now it is so that's probably why I can't reproduce as I fixed this before test and make the video. public function processAfterEdit( $values ) { parent::processAfterEdit( $values ); if ( \IPS\Settings::i()->la_groups_allowed == '*' OR \IPS\Member::loggedIn()->inGroup( explode( ',', \IPS\Settings::i()->la_groups_allowed ) ) AND ( \IPS\Settings::i()->la_groups_postas == '*' OR \IPS\Member::loggedIn()->inGroup( explode( ',', \IPS\Settings::i()->la_groups_postas ) ) ) ) { if ( $this->starter_id !== $values[ 'la_topic_author' ] ) { $this->changeAuthor( \IPS\Member::load( $values[ 'la_topic_author' ] ) ); $post = \IPS\forums\Topic\Post::load( $this->topic_firstpost ); $post->author_id = $this->author()->member_id; $post->author_name = $this->author()->name; $post->save(); } } } Will fix the setting. Tks.
newbie LAC Posted July 25, 2018 Posted July 25, 2018 5 minutes ago, Adriano Faria said: Now it is so that's probably why I can't reproduce as I fixed this before test and make the video. That fix also doesn't work On the video you create/edit topic as one member You need 1 admin account without linked users In this case $values[ 'la_topic_author' ] is null if ( $this->starter_id !== null )
Adriano Faria Posted July 25, 2018 Author Posted July 25, 2018 What's New in Version 3.1.5.1: Fix error when editing first post of topic under certain conditions Thanks to @newbie LAC for clarifying the issue.
Joey_M Posted July 26, 2018 Posted July 26, 2018 I get an error when selecting to post a new topic @Adriano Faria, I have PM'ed it to you. Will reply here. ?
Adriano Faria Posted July 26, 2018 Author Posted July 26, 2018 What's New in Version 3.1.5.2: Fix error when starting a new topic Joey_M 1
Joey_M Posted July 26, 2018 Posted July 26, 2018 Thank you for providing a very quick fix @Adriano Faria, your support is second to none. Cheers! ? Adriano Faria 1
Club Dark Posted August 14, 2018 Posted August 14, 2018 Hi @Adriano Faria I have been experiencing some issues on my 4.3 board with v3.1.5.2. On the Account Settings page I get the following error: Quote [[Template core/front/system/settings is throwing an error. This theme may be out of date. Run the support tool in the AdminCP I tracked the error down to data/theme.xml:123 <template template_group="global" template_name="linkedAccountsGlobalLink" template_data="" template_location="front" template_app="linkedaccounts"><
Recommended Posts