Invision Community 4: SEO, prepare for v5 and dormant account notifications By Matt Monday at 02:04 PM
AnNo1935 Posted July 12, 2018 Posted July 12, 2018 Hey, I have the Blog feature running on my ip board. When users tag each other they do not get a notification like when they tag each other in normal threads. Any idea how to resolve this?
AndyF Posted July 12, 2018 Posted July 12, 2018 First though on this although its not likely it is worth trying, if you are using a custom theme try the tagging on the default one. Does it work now ?
newbie LAC Posted July 12, 2018 Posted July 12, 2018 Hello, It's a funny because I get 2 notifications when someone mention me in the entry. IPS 4.2.4 fresh installation I made quick test and this also happened with downloads files Mentions doesn't work in the Gallery when upload new images. I can mention the user but he don't receive notification Entries and files \applications\blog\sources\Entry\Entry::processForm() and \applications\downloads\sources\File\File::processForm() Both use similar code public function processForm( $values ) { parent::processForm( $values ); if ( !$this->_new ) { $oldContent = $this->content; } $this->content = $values['blog_entry_content']; if ( !$this->_new ) { $this->sendAfterEditNotifications( $oldContent ); } public function processForm( $values ) { parent::processForm( $values ); $new = FALSE; if ( $this->_new ) { $new = TRUE; } else { $oldContent = $this->desc; } $this->desc = $values['file_desc']; if ( !$this->_new ) { $this->sendAfterEditNotifications( $oldContent ); } When you call parent::processForm( $values ); the value of $this->_new that you use bellow is false (if you can use tags) if ( $this instanceof \IPS\Content\Tags and static::canTag( NULL, $this->containerWrapper() ) and isset( $values[ static::$formLangPrefix . 'tags' ] ) ) { $idColumn = static::$databaseColumnId; if ( !$this->$idColumn ) { $this->save(); } So the system send duplicate notifications Tags disabled 1 notification
Daniel F Posted July 13, 2018 Posted July 13, 2018 Thanks, we have fixed this for an upcoming release.
Recommended Posts
Archived
This topic is now archived and is closed to further replies.