search

redbitcz / vyfakturuj-api-php

PHP knihovna pro Vyfakturuj API
https://www.vyfakturuj.cz/api/
MIT License
2 stars 1 forks source link

fix Dvojité require ve wordpressu způsobuje fatal error #3

Closed pavel-janicek closed 6 years ago

pavel-janicek commented 6 years ago

Fix pro moji issue

jakubboucek commented 6 years ago

Related to #2

jakubboucek commented 6 years ago

Pavle, omlouvám se, ale velmi rád bych přijal Tvůj první PR 🎉🎉🎉, ale bohužel to nepovažuju za koncepční řešení.

Dochází-li k chybě, kterou popisuješ v #2, pak je problém spíše v pluginu SimpleShopu pro Wordpress a nebo je tam jiná nekompatibilita, která by se měla řešit tam.

Jinými slovy: Je třeba řešit příčinu chyby a ne jen zalepovat její důsledky.

Toto PR spíše zalepuje důsledky a mohlo by vést k nepředvídatelných chybám (například tím, že by se v jednom projektu skrytě hromadilo více knihoven pro Vyfakturuj, každý v jiné verzi a pak by se aktivoval jen ten, který se načte první).

Z dostupných informací neumím posoudit, co je příčinou chyby. Prosím pomoz mi to zjistit zpřesněním popisu chyb v Issue #2. Díky.

jakubboucek commented 6 years ago

Zavírám bez merge. Nyní připravuji novou verzi a to jak na straně této knihovny, tak na straně SimpleShopu.

Pro úplnost dodávám, že to, co jsi nahlásil, je opravdu v konečném důsledku opravdu chyba ve vzájemné nekompatibilitě těch dvou knihoven. Díky!