Commit Graph

12990 Commits

Author SHA1 Message Date
George Stephanis 516977bd01 JSHint fixes.
The other advantage of splitting js out of php files — jshint can run
on it! :) :) :)
2015-08-07 12:10:01 -04:00
George Stephanis 69b6936d4d Don't constantly rescan the dom for tbody
We have a cached version of $tbody a level up.
2015-08-07 12:01:40 -04:00
George Stephanis 9ae32254f2 Swap `jQuery` to `$`
It’s more readable, and safe to do as we’ve aliased it back in the
enclosure.
2015-08-07 12:00:22 -04:00
George Stephanis c4768e5274 Migrate current_class from php to js 2015-08-07 11:58:16 -04:00
George Stephanis b030927590 Extract 'No Rows Selected' string 2015-08-07 11:58:02 -04:00
George Stephanis bd30def6a1 Whitespace tidy! 2015-08-07 11:57:48 -04:00
George Stephanis 4221c7be24 Extract the autocomplete for states and countries data.
Localize it!
2015-08-07 11:57:38 -04:00
George Stephanis 65eb16fe8a Migrate csv columns to localize script. 2015-08-07 11:47:39 -04:00
George Stephanis 974b337fbb Move js in from php template file. 2015-08-07 11:42:49 -04:00
George Stephanis b0a485a5e6 Change variable name to consistent `$tbody` 2015-08-07 11:41:59 -04:00
George Stephanis 64f48f8706 Rename and tweak how js is registered.
Thanks @claudiosmweb for the suggestion to match existing conventions.
2015-08-07 11:36:38 -04:00
George Stephanis cca93cd09e Prefix template name with `wc-`
Good call, @mikejolley
2015-08-07 11:14:55 -04:00
George Stephanis 729e6c924c Big changeover to JS templating for the table.
Switching over to building the table with JS — wp.template.

We are now using the same JS template for existing rows and newly
generated rows on the fly — this should simplify things moving forward.

I’ve also started building stuff in an external JS file rather than
straight on the page.  Will continue migrating things to it and
localizing as needed.

Saved ( 2 * displayed tax rates - 1 ) db queries per page load by doing
the locations all in one query and then parceling them out in php.

More coming.
2015-08-07 11:04:55 -04:00
James Koster b6c7f14a7b fixes broken extension links 2015-08-07 16:03:45 +01:00
Claudio Sanches 9bc3937689 Updated POT 2015-08-07 10:37:55 -03:00
Claudio Sanches 0a657fdd96 Added missing dot 2015-08-07 10:37:45 -03:00
Claudio Sanches 38aa17b67b Fixed hardcoded string 2015-08-07 10:27:40 -03:00
Claudio Sanches e901114c23 Updated POT 2015-08-07 10:19:08 -03:00
Claudio Sanches c0179c8bb2 Fixed a typo/extra end of line 2015-08-07 10:18:30 -03:00
Mike Jolley 877e390646 Remove $type from prepare 2015-08-07 14:06:19 +01:00
Mike Jolley d6ceec0817 Flip esc_like and esc_sql, add % 2015-08-07 13:54:50 +01:00
Mike Jolley 800ae12552 Merge pull request #8742 from thenbrent/fix_truncated_key
Fix truncated_key value inserted on visual authentication request
2015-08-07 11:45:40 +01:00
Mike Jolley 5c9ac6a893 Avoid double escape when passing in arrays to sql queries. Use esc_sql 2015-08-07 11:37:19 +01:00
Mike Jolley c5cba283ed Remove numbered arguments from queries 2015-08-07 11:37:19 +01:00
Mike Jolley fcbdd35223 Clean and escape API search 2015-08-07 11:37:19 +01:00
Mike Jolley 4c8ef3ff33 Use esc_sql on item type instead of esc_attr 2015-08-07 11:37:19 +01:00
Brent Shepherd 19a91e3afc Fix truncated_key value inserted on visual auth
When inserting a new set of API keys via the visual API endpoint, only 5 format
values were specified in the 3rd param passed to the $wpdb->insert(); however,
6 values were specified in the 2nd param. This meant that the truncted_key value
was being formatted as an integer only the first numerical characters of the last
7 character of the consumer key were being stored. For example, given a consumer
key value of ck_e91f2aeae6c3dea3045293a3dbdf55c317ad762c the truncated_key value
should be 7ad762c but instead, it was being stored at 7.
2015-08-06 17:40:17 -07:00
George Stephanis f0656e7e38 Tax Rate Search: First whack.
This is by no means complete.  A number of things need to be relocated
and optimized still, but it does a rough search, purely in JS, and
highlights the results in yellow.
2015-08-06 17:41:12 -04:00
Nicola Mustone 372d1da118 added new issue link 2015-08-06 18:15:55 +02:00
Nicola Mustone 9d32709f0a typo 2015-08-06 18:09:02 +02:00
Marina Pape 830f93250d Revised copy for 'Description' tab
Warren, Nicole and I have worked on a revised blurb for the description page. It was mostly Nicole, I just did some little tweaks.

Copy in this doc: https://docs.google.com/document/d/1NhaPzGg9I_Esf3N_YqH7ox9s0PiEilEgni_a4DlGqpw/edit

I have edited the existing 'description' as per the google doc, and then added links (as there were none in the doc) for the various extensions mentioned, and a couple of other things that made sense to have links.

Over to you :)
2015-08-06 16:13:33 +01:00
Claudio Sanches c6444c1214 Merge pull request #8737 from SiR-DanieL/master
Contribute to WooCommerce page improvement
2015-08-06 09:25:48 -03:00
Nicola Mustone 3ae67c9cd9 Merge pull request #1 from MarinaWoo/patch-1
Typos, phrasing and grammar edits
2015-08-06 14:12:43 +02:00
Marina Pape d031e97e23 Typos, phrasing and grammar edits
As requested by Nicola, went through and made some edits so standardise grammar / flow etc.
2015-08-06 13:06:00 +01:00
Mike Jolley ddd97b64c3 Remove white space to fix appearance of new rows
Fixes #8730
2015-08-06 11:38:46 +01:00
Mike Jolley b91ccd32b9 Tweak wildcard tax handling to deal with 9 digit zip codes
Fixes #8736
2015-08-06 11:36:02 +01:00
Claudio Sanches 2462fc95d4 Merge pull request #8733 from shivapoudel/escape
Fix - calls for esc_*
2015-08-05 17:31:53 -03:00
Shiva Poudel 0d1c2dc4dc Fix - calls for esc_* 2015-08-06 02:04:21 +05:45
Claudio Sanches cd306393b3 Merge pull request #8732 from woothemes/add/more_esc_attrs
More esc_attrs -- continuation of #8731
2015-08-05 16:54:34 -03:00
Claudio Sanches 5e91d27b66 Minify 2015-08-05 16:52:54 -03:00
Claudio Sanches 6726fd642c Updated grunt dependencies 2015-08-05 16:52:03 -03:00
George Stephanis 9269fd7432 Retooling how this is escaped.
Sanitize early, escape late.  Rather than double escaping, I figured
it’d be better to sprintf it in, and then escape that string — closer
to the output.

Also, I’m not sure why `urlencode` was used to escape a name?
2015-08-05 15:18:47 -04:00
George Stephanis c938fa24da More missing esc_attr calls, :( 2015-08-05 15:17:52 -04:00
Claudio Sanches b091f12393 Fixed some js conding standards 2015-08-05 15:57:15 -03:00
Claudio Sanches 1a2ccd741b Merge pull request #8731 from woothemes/add/esc_attr_e
When echoing translations in an attribute, escape them properly.
2015-08-05 15:55:28 -03:00
George Stephanis 83058779b3 Merge remote-tracking branch 'origin/add/esc_attr_e' into add/esc_attr_e
Conflicts:
	includes/admin/settings/views/html-keys-edit.php
2015-08-05 14:17:18 -04:00
George Stephanis 95e6b84fbb Make sure we esc_attr translation results
Part 2 of … 2, I think.
2015-08-05 14:15:00 -04:00
George Stephanis 29d9a7da8c Found two missing `esc_js()` calls as well. 2015-08-05 14:10:10 -04:00
George Stephanis c7ee929325 Partial _e() => esc_attr_e() changeover.
There’s a lot of them, so I’m breaking them into multiple commits.

This is safeguarding stuff in case some translation uses a double
quote, it will no longer risk breaking out of the attribute.
2015-08-05 14:10:09 -04:00
George Stephanis 883ad9003a Make sure we esc_attr translation results
Part 2 of … 2, I think.
2015-08-05 14:08:15 -04:00