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

[Bugfix] Fix sysfolder workaround with dbal #85

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

Smove
Copy link
Contributor

@Smove Smove commented Aug 23, 2017

Typo3 uses the dbal syntax since v8.4

@dmitryd
Copy link
Owner

dmitryd commented Aug 23, 2017

  1. What does it fix?
  2. It cannot be merged due to wrong formatting of the code.

@Smove
Copy link
Contributor Author

Smove commented Aug 23, 2017

Currently pages doesn't get into sitemap if they are in a sysfolder, the old workaround does not work since dbal is used in core because of the different sql syntax.

I will redo formatting again tomorrow

@Smove
Copy link
Contributor Author

Smove commented Aug 24, 2017

Formatting should be correct now.

@Smove
Copy link
Contributor Author

Smove commented Oct 16, 2017

Any update on this PR?

@echavaillaz
Copy link

Please merge this. It solve my issue : #89

Thanks!

@dmitryd dmitryd merged commit 20ff4fd into dmitryd:development Dec 8, 2017
@dmitryd
Copy link
Owner

dmitryd commented Dec 8, 2017

Thanks!

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

3 participants