Implement missing output escaping

Automattic / babble

Multilingual WordPress done right.

https://wordpress.org/plugins/babble/

245 stars 50 forks source link

Implement missing output escaping #228

Closed mhsdef closed 9 years ago

mhsdef commented 9 years ago

[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L882
[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L888
[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L896
[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L910
[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L918
[x] https://github.com/Automattic/babble/blob/develop/class-jobs.php#L924
[x] https://github.com/Automattic/babble/blob/develop/class-languages.php#L117

More may be added to this post soon.

mhsdef commented 9 years ago

Setup a branch for these fixes here: https://github.com/Automattic/babble/tree/fix/output-escaping

mhsdef commented 9 years ago

More escaping coming later today.

mhsdef commented 9 years ago

Pushed. I think we're ready to go with merging this stuff in after another set of eyes looks at it. I do recommend merging the simplify-hooks PR in first for simplicity.