mirror of
https://github.com/TryGhost/Ghost.git
synced 2024-12-24 19:33:02 +03:00
8d5e7ed695
refs https://github.com/TryGhost/Team/issues/935 The problem was incorrect operator precedence when multiple statements existed in the filter original filter when we transform it to enforce `subscribed:true` before sending. - free only - subscribed:true+status:free - no issue - paid only - subscribed:true+status:-free - no issue - all - subscribed:true+status:-free,status:free - the ,status:free part is treated as a separate OR statement meaning the subscribed:true is not applied to it and free members that are unsubscribed will receive the email - extracted the filter transform into a separate function so it can be unit tested - updated the transform to use `()` for operator precedence, eg: `subscribed:true+(status:-free,status:free)` - used transform function in `addEmail()` and `getEmailMemberRows()` - fixed `sent/send` typo in error message |
||
|---|---|---|
| .. | ||
| adapters | ||
| api | ||
| data | ||
| lib | ||
| models | ||
| public | ||
| services | ||
| views | ||
| web | ||
| analytics-events.js | ||
| ghost-server.js | ||
| notify.js | ||
| overrides.js | ||
| run-update-check.js | ||
| update-check.js | ||