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

Update Standard.php #1210

Closed
wants to merge 1 commit into from
Closed

Conversation

MoniqueHahnefeld
Copy link
Contributor

change path to vendor-folder on Line 79

require_once TL_ROOT . '/vendor/contao/tcpdf/tcpdf.php';

#change path to vendor-folder on Line 79
#require_once TL_ROOT . '/vendor/contao/tcpdf/tcpdf.php';
@aschempp aschempp added the bug label Jul 30, 2014
@aschempp aschempp added this to the 2.2.0 milestone Jul 30, 2014
@Toflar
Copy link
Member

Toflar commented Jul 30, 2014

Dein Ansatz ist leider nicht ganz richtig, aber hab's dank dem Hinweis gefixt, danke :)

@Toflar Toflar closed this Jul 30, 2014
@MoniqueHahnefeld
Copy link
Contributor Author

@Toflar "Dein Ansatz ist leider nicht ganz richtig, aber hab's dank dem Hinweis gefixt, danke :)"

Wieso, die Zeile habe ich doch richtig geändert. Verstehe nicht was du meinst???

@Toflar
Copy link
Member

Toflar commented Jul 31, 2014

Sie muss ganz weg, siegst du ja in meinem Commit :)

@MoniqueHahnefeld
Copy link
Contributor Author

Ahhhhh*mir geht ein Licht auf :) 👍

@Toflar
Copy link
Member

Toflar commented Aug 4, 2014

Allerdings nur für die 2.2.0 (weil die Contao 3.3 bedingen wird). Ansonsten ist der Pfad absolut korrekt. Es dürfte also kein Bug sein in 2.1.4 mit Contao 3.2 :)

@Toflar Toflar modified the milestones: 2.2.0, 2.1.6 Aug 19, 2014
@Toflar
Copy link
Member

Toflar commented Aug 19, 2014

Reopen. 2.1 geht ja auch mit 3.3 :D Müssen also wohl ein version_compare einbauen.

@Toflar Toflar reopened this Aug 19, 2014
@aschempp
Copy link
Member

Fixed for Isotope 2.1 as well

@aschempp aschempp closed this Aug 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants