b62534a6b2
Changeset: Use Maps to simplify attribute processing
2021-11-13 17:44:38 -05:00
9401ae876b
Changeset: Sort attributes by keys, not full string rep
2021-11-13 17:44:38 -05:00
1cad5d881a
Changeset: Use `for...of` iteration to improve readability
2021-11-13 17:44:38 -05:00
6d5b737140
Changeset: Replace `.apply()` with spread operator
2021-11-13 17:44:38 -05:00
9c17b03660
Changeset: Require Op opcode and attribs to be strings
2021-11-13 17:44:38 -05:00
0ae8fb1441
Changeset: Use string concatenation instead of array join
...
People report that string concatenation is faster. Also, I think it's
more readable.
2021-11-13 17:44:38 -05:00
097f2623c6
Changeset: Add sanity checks to `slicerZipperFunc()`
2021-11-13 17:44:38 -05:00
efeb69b4ea
Changeset: Simplify `slicerZipperFunc()`
2021-11-13 17:44:38 -05:00
42d4d8269c
Changeset: Refactor `appendATextToAssembler()` for readability
2021-11-13 17:44:38 -05:00
ca5bdddc59
Changeset: Use `break` instead of `done` variable
2021-11-13 17:44:38 -05:00
44d99733c6
Changeset: Check `.hasNext()` before calling `.next()`
2021-11-13 17:44:38 -05:00
1955e7b263
Changeset: Replace output params with return values
...
This improves readability and reduces the chances of introducing a
bug.
2021-11-13 17:44:38 -05:00
94f5507671
Changeset: Improve `copyOp()` API
...
Use `Object.assign()` to implement `copyOp()`, which simplifies the
code and provides a return value. Also make the second op optional.
2021-11-13 17:44:38 -05:00
02ef78e174
Changeset: Make sure `opOut` is cleared
...
`slicerZipperFunc()` previously assumed the provided `opOut` argument
was a null Op. Enforce this by clearing it at the beginning.
2021-11-13 17:44:38 -05:00
18a6b7279c
Changeset: Only pass strings to `parseNum()`
2021-11-13 17:44:38 -05:00
7ec0d5f385
Changeset: Remove unnecessary `linesApplySplice()`
2021-11-13 17:44:38 -05:00
37bb297e76
Changeset: Improve logged error message
...
I saw this on a production system today and wanted more information.
2021-11-13 17:44:38 -05:00
7fa9b07116
Changeset: Invert conditions to improve readability
2021-11-13 17:44:38 -05:00
b29e59419e
Changeset: Factor out duplicate code
2021-11-13 17:44:38 -05:00
4a65c2c8ff
Changeset: Unexport unnecessarily exported functions
...
These functions aren't used outside of this file.
2021-11-13 17:44:38 -05:00
085bc8cbb3
plugins: Don't create `.ep_initialized` files
...
These files cause problems with Docker images and read-only
directories/mounts, and they have dubious value (any install-time
setup should instead be done at startup).
2021-11-13 17:43:33 -05:00
263105d185
tests: Remove overly aggressive timeouts
2021-11-13 03:05:38 -05:00
3c6aef11bd
lint: Add `'use strict';`
2021-11-13 03:02:40 -05:00
0f7c5a8782
lint: Use `window.` to avoid global declaration
2021-11-12 23:45:05 -05:00
19e9c2f114
adminsettings: Pay attention to `--settings`
2021-11-12 23:45:05 -05:00
46ad93024d
adminsettings: Promisify
2021-11-12 23:45:05 -05:00
4e2f816665
adminsettings: Use object destructuring
2021-11-12 23:45:05 -05:00
b0a7237443
Fix comparison between CustomError and string
2021-11-12 23:40:20 +01:00
6d773f7d56
Put regression tests with the bugfix commit, mention bug in PR
2021-11-10 00:26:10 -05:00
8b73b911c9
Changeset: Use string templates
...
Co-authored-by: Richard Hansen <rhansen@rhansen.org>
2021-11-09 01:05:53 +01:00
cb95e5907c
Localisation updates from https://translatewiki.net .
2021-11-08 13:04:14 +01:00
cc688f7367
ace: Delete dead code
2021-11-07 23:24:39 -05:00
5283bfc14a
linestylefilter: Delete unused arguments
2021-11-07 23:24:39 -05:00
dd8ec4e291
Changeset: Remove unused `lastIndex()` method from op iterator
2021-11-07 23:24:39 -05:00
0fd2a46783
Changeset: Remove unused start index parameter for `opIterator()`
2021-11-07 23:24:39 -05:00
43dae4cb1d
Changeset: Delete unused index parameters in `applyZip()`
2021-11-07 23:24:24 -05:00
b3215992c9
changesettracker: Delete IE8 compatibility code
2021-11-07 22:42:31 -05:00
9f66ffbc3b
Pad: Delete no-op call to `SmartOpAssembler.appendOpWithText()`
2021-11-07 22:42:31 -05:00
ad6de52eb2
Changeset: More JSDoc comment improvements
2021-11-07 22:42:31 -05:00
caefd2184a
PadMessageHandler: Don't fire `userLeave` if user hasn't joined yet
2021-11-07 01:30:39 -04:00
87a490d456
docs: `import` hook `fileEnding` has leading period
2021-11-06 01:05:55 -04:00
088fb14784
fix: upgrade express-rate-limit from 5.4.1 to 5.5.0
...
Snyk has created this PR to upgrade express-rate-limit from 5.4.1 to 5.5.0.
See this package in npm:
https://www.npmjs.com/package/express-rate-limit
See this project in Snyk:
https://app.snyk.io/org/johnmclear/project/d9a12bfb-7ccd-443f-9e22-f30d339cc8c5?utm_source=github&utm_medium=referral&page=upgrade-pr
2021-11-05 18:17:27 -04:00
56cb08f4c5
pad: Pass color and display name in `CLIENT_READY`
2021-11-05 18:14:27 -04:00
7a0d8cb52e
pad: Use URLSearchParams to process search params
...
Also unexport `getUrlVars` because it isn't used outside the file.
2021-11-05 18:14:27 -04:00
fe87e2df87
ImportHandler: Pass the pad ID to the `import` hook
...
This enables plugins to log the pad ID if desired.
2021-11-05 17:19:54 -04:00
caf4e9f28c
ImportHandler: Use truthiness to signal conversion handled
2021-11-05 17:19:54 -04:00
edf99dc811
docs: Document the `import` server-side hook
2021-11-05 17:19:54 -04:00
cc4772a99e
Localisation updates from https://translatewiki.net .
2021-11-04 13:03:08 +01:00
6bf8142221
chat: Translate newly created message entries
2021-11-01 04:56:46 -04:00
99b7bdd176
chat: Set timestamp CSS `font-size: smaller;`
2021-11-01 01:54:29 -04:00
Richard Hansen