Fix a bug that cannot unhide a platform. 
https://bugs.webkit.org/show_bug.cgi?id=203521
Summary Fix a bug that cannot unhide a platform.
dewei_zhu
Reported 2019-10-28 14:26:03 PDT
Fix a bug that cannot unhide a platform.
Attachments
Patch (1.65 KB, patch)
2019-10-28 14:28 PDT, dewei_zhu 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
dewei_zhu
Comment 1 2019-10-28 14:28:24 PDT
Created attachment 382107 [details] Patch
dewei_zhu
Comment 2 2019-10-28 14:30:02 PDT
<rdar://problem/43784654>
Ryosuke Niwa
Comment 3 2019-10-28 14:35:36 PDT
Comment on attachment 382107 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382107&action=review > Websites/perf.webkit.org/public/admin/platforms.php:62 > - || update_field('platforms', 'platform', 'hidden')) > + || update_field('platforms', 'platform', 'hidden', Database::to_database_boolean(array_get($_POST, 'hidden')))) Can we add a helper function like update_boolean_field and do this work there? It's kind of silly to specify the updated field name twice.
Note You need to log in before you can comment on or make changes to this bug.