ExternalChangesList has no documentation on what it's supposed to be and class-orientated (ie everything is static). It also has a 120+ line method that should probably be split up.
Version: master
Severity: normal
Whiteboard: u=dev c=backend p=0
Description
Description
Details
Details
- Reference
- bz42507
Event Timeline
Comment Actions
From the looks of it, it's a utility class for constructing changes entries on on RecentChanges and the watchlist. Unit testing this will probably just be a bunch of regular expressions against the output, but we should have that anyway.
Comment Actions
Yes. Looks like this code has not been improved upon. Also still lacks tests for basically everything.