Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VAT exempt & bugfixes #48

Merged
merged 11 commits into from Jul 24, 2023
Merged

VAT exempt & bugfixes #48

merged 11 commits into from Jul 24, 2023

Conversation

morvy
Copy link
Contributor

@morvy morvy commented Aug 21, 2022

Description & Link to The Issue

Princip odpoctu DPH:

  1. v nastaveniach WooCommerce zaskrtnute nove pole "EU: VAT exempt" (filter woolab_icdic_vat_exempt_enabled)
  2. v nastaveniach WooCommerce ma firma nastavenu svoju domovsku krajinu (filter woolab_icdic_base_country)
  3. nasledne sa odpocet vykona ak su splnene nasledujuce podmienky:
    • domovska krajina sa nezhoduje s krajinou zakaznika (CZ neodpocita CZ, SK neodpocita SK ...)
    • DIC/ICDPH je platne, overene cez VIES

UPDATE:
Druhy commit riesi bug, kde DIČ zobrazovalo duplicitne "optional". Na tomto fielde je uz fixne text, aby ludia vyplnali DIČ len ak su platcovia DPH, takze je vzdy optional.
Treti commit som pridal nejake zakladne informacie pri checkboxe, aby bolo jasne, ze EU VAT Exempt funguje len pre krajinu obchodu v EU a musi mat zapnute dane na eshope. A nakoniec, kedze Woo obsahuje vo WC_Countries spravny zoznam krajin pre odpocet, rovno som to pouzil z Woo jadra namiesto ibericode.

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Design
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

@vyskoczilova
Copy link
Owner

Ahoj, díky moc, musím si to projít - akorát teď dopisuju disertační práci (odevzdávám na konci září) a mám toho nad hlavu. Zkusím to zkontrolovat dřív, ale nerada bych tam udělala nějakou chybu.

Mám v plánu pak minimálně nahradit ibericode za novější závislost, která má lépe ošetřené chyby, ale to určitě až potom (a možná nějaký malý refactor).

@morvy
Copy link
Contributor Author

morvy commented Oct 6, 2022

Ahoj, ako si na tom s casom? Mala si moznost to testnut? Bezi mi to aktualne na 6 weboch, ale neviem, ci je to dostatocne univerzalne ..

@vyskoczilova
Copy link
Owner

Ahoj, omlouvám se - už jsem dopsala, ale chytla jsem covid a jsem KO. Dávám si upomínku na za týden. Moc se omlouvám.

@morvy
Copy link
Contributor Author

morvy commented Nov 20, 2022

Nechcem nejako tlacit, ja len ze moju verziu uspesne pouzivam na par weboch a nehlasia mi ziadne chyby :)

@morvy
Copy link
Contributor Author

morvy commented Jan 17, 2023

Ahoj, chysta sa tu nieco velke, alebo len standardna pracovna vytazenost? :)

@vyskoczilova
Copy link
Owner

Ahoj, pardon, někdy je reálný život složitý.

Zůstávají tu různý otevřený otázky, který chci promyslet dřív, než to pustím ven (protože vždycky skončí na mým e-mailu):

  1. Nejsem úplně jistá tím automatickým zapnutím té VAT funkce - co když se to bude tlouct s nějakýma jinýma pluginama? Možná jenom zobrazit to pole, pokud je tax povolený, ale nechat to vypnutý? Spousta lidí určitě jen tak zaktualizuje plugin a nechci, aby se stalo něco "neočekávanýho".
  2. Tax se mi odečte i v případě, že mám český DIČ (platný, třeba CZ8956190067 - to je moje) a k tomu změním zemi na Slovensko. Musí sedět prefix a země.

Ještě to zkusím protestovat a uvidím, ale asi jinak OK.

P.S. Nemáte nějakou autoritu, vůči který můžu ověřovat IČO? Zdarma?

@morvy
Copy link
Contributor Author

morvy commented Mar 8, 2023

  1. Ja uz si velmi ani nepamatam, co vsetko som tam riesil, no plugin pouzivam na cca 12 weboch. Odpocet DPH sa da checkboxom v admine vypnut, ostane ale kontrola voci VIES. Tu som narazil na problem, ze sa to kontroluje 2x a teda VIES mi na jednom velmi aktivnom eshope blokol IPcku - a teda tam som narazil na dalsi problem, ze pri blokovanom VIES padne cely objednavkovy proces a nejde objednat.

  2. toto je zaujimave, ceske DIC by nemalo byt overitelne voci VIES a teda by nemalo odpocitat DPH pri SK billing country.

  3. na overenie ICO jedine toto: https://github.com/lubosdz/parser-orsr

@vyskoczilova
Copy link
Owner

  1. OK, tak to necháme uložené.

Jak kontroluje dvakrát, jako volá se vies při kontrole a pak při odečítání? Možná by to šlo nějak inteligentně cachovat na serveru třeba na den nebo tak něco (všechno v nějaké jedné proměnné).

Jj, to padne - pokud je pole povinné a neumím ho zkontrolovat, asi by to chtělo pustit dál a přidat tomu nějaké upozornění, ale nikde jsem se k tomu nedostala.

  1. Tak dobrý - sice se mi odebere ihned DPH, ale pokud nemám platné SK dič, tak se objednávka neodešle.

Akorát ještě - pokud jsem plátce DPH, odesílám z Čech na Slovensko a kupuje to neplátce DPH, tak by tam měla Slovenská DPH zůstat, ne? Nejdřív se nic nedělo a teď mne to asi zablokovalo (předpokldám): https://www.loom.com/share/f4617794958144e8aa7590ac6a5cfb6f

nebo?

  1. super, díky (nakonec teda prasácky volám na finstat.sk/IČO a pokud vrátí 200, tak OK, pokud 404 tak platný není, ale tohle se hodí na jindy :-D

@morvy
Copy link
Contributor Author

morvy commented Mar 13, 2023

Ano, kontroluje voci VIES ked dopises DIC, takze overi ci je validne a az potom oznaci objednavku is_vat_exempt. Nasledne ale este kontroluje voci VIES, ked sa spracuva objednavka a teda ci je DIC validne (to je vlastne to, co tam bolo doteraz). Ja som rozmyslal dat tam na 1 hodinu transient ak je DIC validne (a najprv musi byt spravny format, kym nema spravny format, neposiela ani do VIES).

To spravanie na videu si myslim, ze je prave blok zo strany VIES. Ak by si cez Postman skusila tu WSDL url, najskor by si dostala 401 alebo nieco take. Ale celkovo to spravanie, ktore popisujes sa mi neprejavuje ked to testujem ja, ale viem, ze vo vynimocnych pripadoch niekomu odpocitalo DPH aj ked nemalo. Plugin pouzivam na SK, CZ, HU, PL eshopoch kde je SK firma, takze v CZ odpocita ak niekto zada DIC a tak aj dalsie krajiny, ale na SK neodpocita SK a ani v CZ neodpocita pre SK fakturacnu.

Ohladom Finstat .. cool lifehack, ale neviem ci vracia 200 alebo iny status pre firmy, ktore sice este su na Finstat, ale uz su nefunkcne.

@morvy
Copy link
Contributor Author

morvy commented Mar 15, 2023

@vyskoczilova este ma k 4. bodu osvietilo, ze mame toto API, len co viem, je dost limitovane a treba to intenzivne cachovat

https://api.statistics.sk/rpo/v1/search?identifier=123456789&showHistoricalData=true

morvy and others added 2 commits April 16, 2023 19:00
Added filter to modify the exempt value for use in set_is_vat_exempt. Handy for B2B shops and ROW where no VAT is paid.
@vyskoczilova
Copy link
Owner

Ahoj @morvy,
omlouvám se za všechna zdržení, funguje, jak má, merguju (a projdu ostatní pullrequesty a pustím to ven rovnou).
K.

@vyskoczilova vyskoczilova merged commit fd90e90 into vyskoczilova:master Jul 24, 2023
vyskoczilova added a commit to morvy/kybernaut-ic-dic that referenced this pull request Jul 24, 2023
* VAT exempt & bugfixes
* fix duplicate "optional" on DIC field
* use WC core for VAT countries, more info for user in Settings
* Check VAT num format before checking against VIES
* is_vat_exempt filters
Added filter to modify the exempt value for use in set_is_vat_exempt. Handy for B2B shops and ROW where no VAT is paid.
* composer
* Add billing ic/dic only on order edit screen.
* update language file

---------

Co-authored-by: Karolína Vyskočilová <karolina@kybernaut.cz>
vyskoczilova added a commit that referenced this pull request Jul 24, 2023
* Improve DIC/ICDPH aka CZ/SK VAT validation

* composer

* Add billing ic/dic only on order edit screen.

* VAT exempt & bugfixes (#48)

* VAT exempt & bugfixes
* fix duplicate "optional" on DIC field
* use WC core for VAT countries, more info for user in Settings
* Check VAT num format before checking against VIES
* is_vat_exempt filters
Added filter to modify the exempt value for use in set_is_vat_exempt. Handy for B2B shops and ROW where no VAT is paid.
* composer
* Add billing ic/dic only on order edit screen.
* update language file

---------

Co-authored-by: Karolína Vyskočilová <karolina@kybernaut.cz>

* add readme

---------

Co-authored-by: Karolína Vyskočilová <karolina@kybernaut.cz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants