Adri Oosterwijk

My feedback

  1. 3 votes
    Vote
    Sign in
    (thinking…)
    Password icon
    Signed in as (Sign out)
    You have left! (?) (thinking…)
    Adri Oosterwijk commented  · 

    Anything new to mention about this topic?

    Adri Oosterwijk commented  · 

    Same for the following:

    In class-wc-ta-request.php are those lines (247-250):

    $error_message = apply_filters( 'woocommerce_taxamo_validation_error_message_location', "We couldn't confirm your location." );

    if ( 'yes' === WC_TA_Integration::$enable_self_declaration ) {
    $error_message .= apply_filters( 'woocommerce_taxamo_validation_error_message_manually_validate', " Please manually validate it." );

    I had to change it to:

    $error_message = apply_filters( 'woocommerce_taxamo_validation_error_message_location', __("We couldn't confirm your location." , 'woocommerce-taxamo' ) );

    if ( 'yes' === WC_TA_Integration::$enable_self_declaration ) {
    $error_message .= apply_filters( 'woocommerce_taxamo_validation_error_message_manually_validate', __(" Please manually validate it." , woocommerce-taxamo) );

    to get it translatable.

    Please work with priority on a sustainable solution.

    Regards,

    Adri

    Adri Oosterwijk commented  · 

    Same for: the lines 74 and 75 in the same file:

    // Add the new tax rate for this transaction line
    $tax_manager->add_tax_rate( $tl->product_type, 0.00, 'European VAT' );

    Change it to:

    // Add the new tax rate for this transaction line
    $tax_manager->add_tax_rate( $tl->product_type, 0.00, __('European VAT' , 'woocommerce-taxamo') );

    I have read somewhere that it is not a very reliable solution to embed gettext statements in variables but this is the kind it should work. I trust that you are able to find a reliable solution for this if the above turns out to be not so. This applies also to my last comment.

    Adri Oosterwijk commented  · 

    Went digging again and it is in: class-wc-ta-checkout-vat.pho

    On line 88 and 89 you see:
    // Add the new tax rate for this transaction line
    $tax_manager->add_tax_rate( $tl->product_type, $tl->tax_rate, strtoupper( 'VAT ' . $country_code . ' ' . $tl->product_type ) );

    the text 'VAT' is not translatable.
    I changed it to:

    // Add the new tax rate for this transaction line
    $tax_manager->add_tax_rate( $tl->product_type, $tl->tax_rate, strtoupper( __('VAT', 'woocommerce-taxamo') . ' ' . $country_code . ' ' . $tl->product_type ) );

    Because it is not a template file I can't add it to my child theme. That means with every update of the Taxamo plugin this customization is lost. It would be great when could be in the standard file.

    Cheers!

    Adri Oosterwijk commented  · 

    mmm must missed your message.

    I don't use .mo files, I'm using WPML to translate strings to languages. The main "problem" is the VAT message on the checkout page and the invoice. It displays the text "VAT NL E-SERVICE" where I want to display "BTW NL E-SERVICE" for Dutch (and of course the right translations for any other language I plan to add to the site.

    Adri

    Adri Oosterwijk supported this idea  · 
    Adri Oosterwijk shared this idea  · 
  2. 126 votes
    Vote
    Sign in
    (thinking…)
    Password icon
    Signed in as (Sign out)
    You have left! (?) (thinking…)
    17 comments  ·  WooCommerce » Category - Extension Ideas  ·  Flag idea as inappropriate…  ·  Admin →
    Adri Oosterwijk supported this idea  · 
    Adri Oosterwijk commented  · 

    Yes! I want this as well. It must be simple. The VAT numbers are stored in the table: wp_postmeta
    meta_key: vat_number
    meta_value: (the vat number)
    So why not store it in the customer table as well...? Please be quick with it.

  3. 9 votes
    Vote
    Sign in
    (thinking…)
    Password icon
    Signed in as (Sign out)
    You have left! (?) (thinking…)
    1 comment  ·  WooCommerce » Category - Extension Ideas  ·  Flag idea as inappropriate…  ·  Admin →
    Adri Oosterwijk supported this idea  · 

Feedback and Knowledge Base