r93967 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r93966‎ | r93967 | r93968 >
Date:13:27, 5 August 2011
Author:devayon
Status:deferred
Tags:
Comment:
Follow-up: r93878
Modified paths:
  • /trunk/extensions/SemanticMediaWiki/specials/AskSpecial/SMW_SpecialQueryCreator.php (modified) (history)

Diff [purge]

Index: trunk/extensions/SemanticMediaWiki/specials/AskSpecial/SMW_SpecialQueryCreator.php
@@ -65,12 +65,12 @@
6666 $result .= '<br>';
6767 $result .= wfMsg( 'smw_qc_query_help' );
6868 // Main query and format options
69 - $result .= '<table style="width: 100%; ">'.
70 - '<tr><th>' . wfMsg( 'smw_ask_queryhead' ) . "</th>\n<th>" . wfMsg( 'smw_ask_format_as' ) . "</th></tr>".
71 - '<tr>'.
72 - '<td style="padding-right: 7px;">'.$this->getQueryFormBox()."</td>\n" .
73 - '<td style="padding-right: 7px; text-align:center;">'. $formatBox[0] .'</td>'.
74 - '</tr>'.
 69+ $result .= '<table style="width: 100%; ">' .
 70+ '<tr><th>' . wfMsg( 'smw_ask_queryhead' ) . "</th>\n<th>" . wfMsg( 'smw_ask_format_as' ) . "</th></tr>" .
 71+ '<tr>' .
 72+ '<td style="width: 70%; padding-right: 7px;">' . $this->getQueryFormBox() . "</td>\n" .
 73+ '<td style="padding-right: 7px; text-align:center;">' . $formatBox[0] . '</td>' .
 74+ '</tr>' .
7575 "</table>\n";
7676 // sorting and prinouts
7777 $result .= $this->getPoSortFormBox();
@@ -89,7 +89,7 @@
9090 // additional options
9191 $result .= '<div id="additional_options" style="display:none">';
9292
93 - $result .= $formatBox[1]; //display the format options
 93+ $result .= $formatBox[1]; // display the format options
9494
9595 if ( $this->uiCore->getQueryString() != '' ) // hide #ask if there isnt any query defined
9696 $result .= $this->getAskEmbedBox();
@@ -188,26 +188,26 @@
189189 $result .= Html::openElement( 'select', array( 'name' => "order[$i]" ) );
190190
191191 $if1 = ( !is_array( $order_values ) or !array_key_exists( $i, $order_values ) or $order_values[$i] == 'NONE' );
192 - $result .= Xml::option(wfMsg( 'smw_qui_nosort' ), "NONE", $if1);
 192+ $result .= Xml::option( wfMsg( 'smw_qui_nosort' ), "NONE", $if1 );
193193
194194 $if2 = ( is_array( $order_values ) and array_key_exists( $i, $order_values ) and $order_values[$i] == 'ASC' );
195 - $result .= Xml::option(wfMsg( 'smw_qui_ascorder' ), "ASC", $if2);
 195+ $result .= Xml::option( wfMsg( 'smw_qui_ascorder' ), "ASC", $if2 );
196196
197197 $if3 = ( is_array( $order_values ) and array_key_exists( $i, $order_values ) and $order_values[$i] == 'DESC' );
198 - $result .= Xml::option(wfMsg( 'smw_qui_descorder' ), "DESC", $if3);
 198+ $result .= Xml::option( wfMsg( 'smw_qui_descorder' ), "DESC", $if3 );
199199
200 - $result .= Xml::closeElement('select');
 200+ $result .= Xml::closeElement( 'select' );
201201
202202 $if4 = ( is_array( $display_values ) and array_key_exists( $i, $display_values ) );
203 - $result .=Xml::checkLabel(wfMsg( 'smw_qui_shownresults' ), "display[$i]", "display$i", $if4 );
 203+ $result .= Xml::checkLabel( wfMsg( 'smw_qui_shownresults' ), "display[$i]", "display$i", $if4 );
204204
205 - $result .= Xml::closeElement('div');
 205+ $result .= Xml::closeElement( 'div' );
206206 }
207207 // END: create form elements already submitted earlier via form
208208
209209 // create hidden form elements to be cloned later
210210 $hidden = Html::openElement( 'div', array( 'id' => 'sorting_starter', 'style' => 'display:none' ) ) .
211 - '<span class="smw-remove"><a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</a></span>'.
 211+ '<span class="smw-remove"><a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</a></span>' .
212212 wfMsg( 'smw_qui_property' ) .
213213 Xml::input( "property_num", '35' ) . " ";
214214
@@ -222,30 +222,34 @@
223223
224224 $hidden = json_encode( $hidden );
225225
226 - $dialogbox = Xml::openElement('div', array('id'=>'dialog')) .
227 - Xml::checkLabel(wfMsg( 'smw_qui_shownresults' ), '', 'dialog-show-results', true).
228 - '<div id="tab-box">'.
229 - '<ul>'.
230 - '<li><a href="#property-tab">Property</a></li>'. //todo i18n
231 - '<li><a href="#category-tab">Category</a></li>'. //todo i18n
232 - '</ul>'.
233 - '<div id="property-tab">'.
234 - Xml::inputLabel('Property', '','tab-property', 'tab-property'). '<br/>'. //todo i18n
235 - Xml::inputLabel('Label (optional):', '','tab-property-label', 'tab-property-label'). '<br/>'. //todo i18n
236 - 'Format: '. Html::openElement('select', array('name'=>'tab-format')) . //todo i18n
237 - Xml::option('None (default)', 'NONE'). //todo i18n
238 - Xml::option('Simple', '-'). //todo i18n
239 - Xml::option('Numeric', 'n'). //todo i18n
240 - Xml::option('Unit', 'u'). //todo i18n
241 - Xml::option('Custom', 'CUSTOM'). //todo i18n
242 - Xml::closeElement('select').
243 - Xml::input('format-custom').
244 - '</div>'.
245 - '<div id="category-tab">'.
246 - 'category options go here'.
247 - '</div>'.
248 - '</div>'.
249 - '<br>Sort by: <select id ="dialog-order">' . //todo i18n
 226+ $dialogbox = Xml::openElement( 'div', array( 'id' => 'dialog', 'title' => 'Advanced Print-Out Options' ) ) . // todo i18n
 227+ Xml::checkLabel( wfMsg( 'smw_qui_shownresults' ), '', 'dialog-show-results', true ) .
 228+ '<div id="tab-box">' .
 229+ '<ul>' .
 230+ '<li><a href="#property-tab">Property</a></li>' . // todo i18n
 231+ '<li><a href="#category-tab">Category</a></li>' . // todo i18n
 232+ '</ul>' .
 233+ '<div id="property-tab">' .
 234+ Xml::inputLabel( 'Property', '', 'tab-property', 'tab-property' ) . '<br/>' . // todo i18n
 235+ Xml::inputLabel( 'Label (optional):', '', 'tab-property-label', 'tab-property-label' ) . '<br/>' . // todo i18n
 236+ 'Format: ' . Html::openElement( 'select', array( 'name' => 'tab-format' ) ) . // todo i18n
 237+ Xml::option( 'None (default)', 'NONE' ) . // todo i18n
 238+ Xml::option( 'Simple', '-' ) . // todo i18n
 239+ Xml::option( 'Numeric', 'n' ) . // todo i18n
 240+ Xml::option( 'Unit', 'u' ) . // todo i18n
 241+ Xml::option( 'Custom', 'CUSTOM' ) . // todo i18n
 242+ Xml::closeElement( 'select' ) .
 243+ Xml::input( 'format-custom' ) . '<br/>' .
 244+ Xml::inputLabel( 'limit (optional):', '', 'tab-property-limit', 'tab-property-limit' ) . '<br/>' . // todo i18n
 245+ '</div>' .
 246+ '<div id="category-tab">' .
 247+ Xml::inputLabel( 'Label (optional):', '', 'tab-category-label', 'tab-category-label' ) . '<br/>' . // todo i18n
 248+ Xml::inputLabel( 'Specify a category (optional)', '', 'tab-category', 'tab-category' ) . '<br/>' . // todo i18n
 249+ 'If result belongs to category, display' . Html::input( 'tab-yes', 'X' ) . '<br/>' .
 250+ 'else display' . Html::input( 'tab-yes', ' ' ) .
 251+ '</div>' .
 252+ '</div>' .
 253+ '<br>Sort by: <select id ="dialog-order">' . // todo i18n
250254 '<option value="NONE">' . wfMsg( 'smw_qui_nosort' ) . '</option>' .
251255 '<option value="ASC">' . wfMsg( 'smw_qui_ascorder' ) . '</option>' .
252256 '<option value="DESC">' . wfMsg( 'smw_qui_descorder' ) . '</option>' .
@@ -269,7 +273,7 @@
270274 autoOpen: false,
271275 modal: true,
272276 resizable: true,
273 - minHeight: 500,
 277+ minHeight: 300,
274278 minWidth: 500
275279 });
276280 jQuery('#tab-box').tabs({

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r93878adding modal window for sorting optionsdevayon10:44, 4 August 2011