Possible copy&paste mistake in Discourse User Model Code

Moin · August 1, 2024, 10:39am

I have the impression that silencedTill in line 401 and 402 should be suspendedTill

discourse/discourse/blob/608e5682e2e706fec20fbe4b82b8b95cb962072d/app/assets/javascripts/discourse/app/models/user.js#L390-L408


      
          @discourseComputed("suspended_till")
          suspendedForever(suspendedTill) {
            return isForever(suspendedTill);
          }
          
          @discourseComputed("silenced_till")
          silencedForever(silencedTill) {
            isForever(silencedTill);
          }
          
          @discourseComputed("suspended_till")
          suspendedTillDate(silencedTill) {
            return longDate(silencedTill);
          }
          
          @discourseComputed("silenced_till")
          silencedTillDate(silencedTill) {
            return longDate(silencedTill);
          }

joffreyjaffeux · August 1, 2024, 1:33pm

Yes you are correct, however this is not impacting behavior, just the name is incorrect.

That will do it:

Firepup650 · August 1, 2024, 1:35pm

Shouldn’t line 397 be a return statement as well?

joffreyjaffeux · August 1, 2024, 1:50pm

Yes it should indeed, that’s a behavior change, a mistake we did in a refactoring few months ago.

Topic		Replies	Views
Typo in suspension reasons list UX	2	460	May 18, 2021
Wrong custom date and time in silenced and suspended warnings Bug	5	554	May 31, 2022
Permissions buttons not working Support	4	948	December 20, 2016
[missing {{tarih}} value] Bug	4	552	May 28, 2018
Undefined method `scan' Translations	3	857	March 6, 2019

Possible copy&paste mistake in Discourse User Model Code

Related Topics