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

Datumsfeld "strDateField" in File Sales.php updatesicher zur Verfügung stellen #1620

Closed
error08 opened this issue Mar 6, 2016 · 6 comments
Labels
Milestone

Comments

@error08
Copy link
Contributor

error08 commented Mar 6, 2016

Kann das Feld $strDateField aus der Klasse Sales updatesicher als konfigurierbarer Wert entweder in den Config-Files oder im Backend zur Verfügung gestellt werden??

// Allow extensions to use date_paid or date_shipped protected $strDateField = 'locked';

@aschempp aschempp added this to the 2.x.x milestone Mar 7, 2016
@aschempp
Copy link
Member

aschempp commented Mar 7, 2016

Was möchtest du damit erreichen?

@error08
Copy link
Contributor Author

error08 commented Mar 7, 2016

Das die Einstellung bei einem Versionsupdate von Isotope nicht überschrieben wird. Ich habe das Datum für einen Kundenshop auf date_paid eingestellt. Dies sollte mit dem nächsten Update nicht wieder überschrieben werden.

@aschempp aschempp modified the milestones: 2.4.0, 2.x.x Mar 10, 2016
@error08
Copy link
Contributor Author

error08 commented Mar 12, 2016

Hierzu müsste natürlich auch die Where-Bedingung in den Auswertungen angepasst werden. Dies ist mir erstmal auf die Füße gefallen :)
WHERE o.type='order' AND o.order_status>0 AND o.$this->strDateField!=''

@aschempp
Copy link
Member

Inwiefern müsste die WHERE-Bedingung angepasst werden? Die Idee des protected Feldes ist eigentlich dass man seinen eigenen Report (eigene PHP-Klasse) erstellt und das entsprechend umkonfigurieren könnte.

@error08
Copy link
Contributor Author

error08 commented Apr 20, 2016

In meinen Fall soll das Bezugsdatum nicht das Feld locked sonder date_paid sein. Da dies für alle Sales-Reports gelten soll,war meine Idee dieses Feld updatesicher ändern zu können.
Das es funktioniert müsste im SQL-Statement die Stellen mit dem Feld locked durch die Variable ersetzt werden.
Wenn der Sinn war,die vorhanden Reports zu kopieren,um das eine Feld zu ändern kann ich auch das machen. So würde man die Weiterentwicklung der Orginalreports ggf. nicht mitbekommen.

@aschempp
Copy link
Member

In 2.4 gibt es nun ein Dropdown zur Auswahl des Report-Feldes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants