Skip to content

</form> in form class #5

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

Merged
merged 2 commits into from
Jun 14, 2011
Merged

</form> in form class #5

merged 2 commits into from
Jun 14, 2011

Conversation

mikelbring
Copy link
Contributor

My IDE or any editor like's to pretend there are errors when I use the form open method but then put an html </form>. I figured this would fix that problem. You open the form with PHP, why not close it with PHP.

I did this on your develop branch. Not sure if thats the best practice when sending pull requests. Never done this before.

@taylorotwell
Copy link
Member

Thanks! I'll merge this in later today.

@pedroborges
Copy link
Contributor

@taylor, is it needed to have PHP_EOL at the end of the string in this case?

@mikelbring
Copy link
Contributor Author

Didn't think about that. Do you want me to update it?

@pedroborges
Copy link
Contributor

Well, I came to know PHP_EOL reading Laravel's code. Since it's newline character that is cross-platform-compatible I think it's to good to have it. Go ahead!

@taylorotwell
Copy link
Member

Yeah go ahead and put one. Thanks!

On Jun 14, 2011, at 11:27 AM, pedroborgesreply@reply.github.com wrote:

@taylor, is it needed to have PHP_EOL at the end of the string in this case?

Reply to this email directly or view it on GitHub:
https://github.com/taylorotwell/laravel/pull/5#issuecomment-1367213

@mikelbring
Copy link
Contributor Author

I did, see my previous commit ^^

taylorotwell added a commit that referenced this pull request Jun 14, 2011
@taylorotwell taylorotwell merged commit e87f7dc into laravel:develop Jun 14, 2011
taylorotwell pushed a commit that referenced this pull request Jan 5, 2013
flap152 pushed a commit to flap152/laravel that referenced this pull request Nov 24, 2017
flap152 pushed a commit to flap152/laravel that referenced this pull request Nov 24, 2017
phithienthan referenced this pull request in phithienthan/laravel Apr 18, 2018
sl0wik added a commit to sl0wik/laravel that referenced this pull request Oct 4, 2023
flap152 pushed a commit to flap152/laravel that referenced this pull request Jan 29, 2024
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.

4 participants