Page MenuHomePhabricator

[native] Default parsedSpoilerIdentifier to null
ClosedPublic

Authored by rohan on Dec 7 2022, 9:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 11:50 AM
Unknown Object (File)
Sat, Nov 9, 11:50 AM
Unknown Object (File)
Sat, Nov 9, 11:49 AM
Unknown Object (File)
Fri, Nov 8, 9:59 AM
Unknown Object (File)
Thu, Nov 7, 5:47 AM
Unknown Object (File)
Thu, Nov 7, 5:47 AM
Unknown Object (File)
Wed, Nov 6, 6:22 AM
Unknown Object (File)
Tue, Nov 5, 2:08 AM
Subscribers

Details

Summary

Not a big deal, but @atul and I decided that we should default parsedSpoilerIdentifier to null in the event of an unexpected value. Note that I still need to explicity check if parsedSpoilerIdentifier !== null, since if I just checked if (parsedSpoilerIdentifier) and parsedSpoilerIdentifier = 0, it would make the condition false.

https://linear.app/comm/issue/ENG-2269/investigate-if-defaulting-parsedspoilerindex-to-1-is-the-best-solution

Test Plan

Tested both spoilers mid-text and at the start of the text to make sure they still work as expected.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable