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

Fall back to index.php if file not found. #10210

Merged
merged 1 commit into from
May 3, 2016

Conversation

Bakual
Copy link
Contributor

@Bakual Bakual commented May 3, 2016

Pull Request for Issue #10207 .

Summary of Changes

This PR simply adds a fallback to index.php in case the requested file isn't present neither in the active template nor in the system template

Testing Instructions

Test by adding ?tmpl= or ?tmpl=foo to the URL. Without this PR you will get a blank page, with this PR the page will be rendered with the system/index,php file (looks ugly but it renders).

@Bakual Bakual mentioned this pull request May 3, 2016
@andrepereiradasilva
Copy link
Contributor

I have tested this item ✅ successfully on 739d22b

looks ugly but it renders


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10210.

@infograf768
Copy link
Member

I have tested this item ✅ successfully on 739d22b


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10210.

@infograf768
Copy link
Member

RTC.Thanks.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10210.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 3, 2016
@brianteeman brianteeman added this to the Joomla! 3.6.0 milestone May 3, 2016
@rdeutz rdeutz merged commit 2e4cfe2 into joomla:staging May 3, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 3, 2016
@Bakual Bakual deleted the FixThemefileNotPresent branch May 3, 2016 10:09
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

6 participants