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

zasadniczy błąd przy działaniu pola 'odbiorca faktury' #28

Open
byq opened this issue Feb 12, 2013 · 5 comments
Open

zasadniczy błąd przy działaniu pola 'odbiorca faktury' #28

byq opened this issue Feb 12, 2013 · 5 comments

Comments

@byq
Copy link
Contributor

byq commented Feb 12, 2013

pole 'odbiorca faktury' nie może działać tak jak obecnie, czyli tak jak pole 'adres do wysyłki', czyli że jedną zmianą na karcie klienta zmieniamy odbiorcę we wszystkich wystawionych już dokumentach!

Podczas tworzenia faktury, lub podczas jej edycji dane odbiorcy muszą być brane z 'odbiorca faktury', a jeżeli = NULL then z 'główny adres'

np:
podpisałem umowę z klientem A, za miesiąc zmienił nazwisko na B. Zmianę dokonuję w polu odbiorca faktury i część faktur ma zostać z odbiorcą A, a wszystkie nowe z B.
Obecnie wszystkie zmienią na B.

@byq
Copy link
Contributor Author

byq commented Feb 12, 2013

do zmiany w LMS.class.php funkcja AddInvoice($invoice)
musi pojawic się if sprawdzajacy czy $invoice['customer']['invoice_name'] && $invoice['customer']['invoice_address'] && $invoice['customer']['invoice_zip'] są równe NULL
i if not to brać z tych miejsc właśnie.

podobnie w invoiceedit.php linia 227 $DB->Execute('UPDATE documents

i w skryptach lms-payments

przechodziłem robienie takich zmian na lms 1.10.4

trzeba też odkręcić zmiany przy generowaniu faktury.

@byq
Copy link
Contributor Author

byq commented Feb 12, 2013

myślę, że można uprościć if'a do sprawdzania tylko czy jest invoice_name

i przy okazji: obecnie jest tak, że można wpisać w adresie odbiorcy tylko name i formularz to przyjmie, ale już invoce.php generując pdf'a sprawdza czy są wypełnione następne pola (adres, zip) i if not to podstawia adres główny.

@inetlms
Copy link
Owner

inetlms commented Feb 12, 2013

to co opisujesz, że zmieniasz jedną rzecz i wszystkie faktury wstecz też się zmieniają to jest odziedziczone po LMS 1.11-git :(
Jeżeli chodzi o sam temat faktur, ich wystawiania, trzymania itp to będzie to przerobione, i wtedy "błędy" o których piszesz będą mogły być wyeliminowane. Na dzień dzisiejszy niestety to musi pozostać tak jak jest :/

@byq
Copy link
Contributor Author

byq commented Feb 12, 2013

e tam musi...
nie ma wiele roboty przy tym. Moja produkcyjna wersja lms'a 1.10.4 ma to zaimplemetnowane i działa. Pogrzebie i zrobie pull. Sam ocenisz.

@byq
Copy link
Contributor Author

byq commented Feb 19, 2013

zacząłem tworzyć...
https://github.com/byq/lms-1

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

No branches or pull requests

2 participants