[wp-meta] [Making WordPress.org] #558: CampTix "Country" question type
Making WordPress.org
noreply at wordpress.org
Thu Jan 1 17:32:40 UTC 2015
#558: CampTix "Country" question type
-------------------------+-------------------------------------------------
Reporter: | Owner:
vanillalounge | Status: new
Type: | Component: wordcamp.org
enhancement | Keywords: good-first-bug reporter-feedback
Priority: normal | needs-testing
Resolution: |
-------------------------+-------------------------------------------------
Comment (by iandunn):
Hey Deven, thanks for the patch :)
Overall it looks good, but there are a few things I think we should
change.
1. We don't need to show the list of countries in the back-end, or save it
in the `postmeta` for every `tix_question`, we can just output the
hardcoded list on the front-end for the visitor, similar to how the
Twitter and URL fields don't ask for anything.
1. `question_field_country()`, etc should be in `addons/field-country.php`
rather than `camptix.php`, because addons shouldn't modify core CampTix
(just like WordPress plugins don't modify Core files). It's fine to add it
to `$default_addons`, though.
1. The patch should be compliant with
[http://make.wordpress.org/core/handbook/coding-standards/ WP's coding
standards] (braces around single lines, etc).
1. Escape output as late as possible, e.g., during the `printf` rather
than the `$country = ...`
1. Remove debugging code like `// echo $country;`
Also, where did the list of countries come from? We'll want to make sure
it matches the official ISO 3166-1 list.
--
Ticket URL: <https://meta.trac.wordpress.org/ticket/558#comment:3>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org
More information about the wp-meta
mailing list