NodeBB / NodeBB / 27551108045

85%

push

github

fix: bind hooks module in admin/settings/email define() (#14352)

The define() call lists four dependencies but the factory only declared
three parameters, so `hooks` resolved to the `admin/settings` module and
`hooks.onPage` threw, breaking the entire admin email settings page
(fields no longer populate from saved config). Add the missing `settings`
parameter so each dependency binds to the correct module.

14051 of 19198 branches covered (73.19%)

29392 of 34414 relevant lines covered (85.41%)

3475.81 hits per line