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

fix docs #967

Closed
wants to merge 1 commit into from
Closed

fix docs #967

wants to merge 1 commit into from

Conversation

ambs
Copy link
Member

@ambs ambs commented Jul 29, 2015

the previous documentation used '$filename' as argument to upload, but it should be the field name.

@ambs
Copy link
Member Author

ambs commented Aug 2, 2015

Can I merge this one? It is docs, so, no big deal, but would like a 👍

@veryrusty
Copy link
Member

I did wonder if the param file and the form field file may cause confusion?
Possibly name_of_upload_input (or something less verbose) would be better.

However, I don't want to discourage correct docs, so merge away if you don't think its confusing.

@ambs
Copy link
Member Author

ambs commented Aug 5, 2015

But aren't all form fields parameters?
That is, file is the same field in the two lines. But if you use the param method, you get the filename, but if you query the upload object, you get an object. Right?

But I can add some text to make it clear, if you like ;)

@veryrusty
Copy link
Member

There are two params named file in the example; one is a route param, the other is a body param (form field).

@ambs
Copy link
Member Author

ambs commented Aug 6, 2015

And you say you are rusty :-P sorry, and thanks for pointing me to the route line, I didn't look to it with caution. probably I can change the upload route and remove the parameter, that is "useless" for the example. Anything against it?

@veryrusty
Copy link
Member

@ambs Feel free to totally rework the example :)

@ambs
Copy link
Member Author

ambs commented Aug 8, 2015

@veryrusty, minor change. In vacations, so, trying to work the less possible O:-)
Would this be good enough for now? :-)

@veryrusty
Copy link
Member

@ambs that still not right; $filename isn't defined. Enjoy your vacation and I'll fix it ;)

@veryrusty
Copy link
Member

Merged as c848047. Thanks @ambs! 👯

@veryrusty veryrusty closed this Aug 9, 2015
@ambs
Copy link
Member Author

ambs commented Aug 9, 2015

Gah, stupid me.
Thanks and sorry for the incompetence.
Alberto

-----Original Message-----
From: Russell Jenkins notifications@github.com
To: PerlDancer/Dancer2 Dancer2@noreply.github.com
Cc: "Alberto Simões" ambs@cpan.org
Sent: Sun, 09 Aug 2015 12:25
Subject: Re: [Dancer2] fix docs (#967)

@ambs that still not right; $filename isn't defined. Enjoy your vacation and I'll fix it ;)


Reply to this email directly or view it on GitHub:
#967 (comment)

xsawyerx added a commit that referenced this pull request Aug 28, 2015
    [ BUG FIXES ]
    * GH #947, #948: Escape file paths in regex patterns. (A. Sinan Unur)
    * GH #944: Setting response content in before hook when a serializer
      is set no longer triggers an error.
      (Russell Jenkins, Dmitrii Tcyganov)
    * GH #965: Remove non-existant role from Response::Delayed.
      (Vernon, Russell Jenkins)
    * GH #971: Route options matching no longer uses each iterator.
      (Tina Müller)
    * GH #959: Custom error template rendering fixed. (Russell Jenkins)
    * GH #961: Render custom error templates in before hooks. (Russell Jenkins)
    * GH #978: Tests - fix response regex after html_encode (Vernon)
    * GH #972: Exceptions thrown by serializers no longer masked.
      (Russell Jenkins)

    [ DOCUMENTATION ]
    * GH #967: Fix upload example. (Alberto Simões)
    * GH #881: Add cookie timeout example. (Andy Beverley)
    * GH #963: Document all available template tokens. (Sawyer X)

    [ ENHANCEMENTS ]
    * Optimize the s*#t out of basic routing. Faster than Dancer 1 now.
      (Sawyer X)
    * Only load HTTP::Server::PSGI when asked to start a development
      server not under Plack. (Sawyer X, Mickey Nasriachi)
    * GH #949: Produce cleaner, non-verbose test output (Vernon)
    * GH #950: Decode characters in param keys (Patrick Zimmermann)
    * GH #914: Include stack trace on default error page when
      show_errors is true. (Vernon)
    * GH #980, #981: halt keyword sets response content if provided,
      as Dancer 1 does. (Achilles Kars)
    * GH #909, #957, #983: HTML5 templates in generated apps and
      default error template (Gabor Szabo, Kadir, Vernon)
    * GH #972, #719, #969, #644, #647: Streamline serializer helpers.
      to_json/from_json now faster. (Russell Jenkins)
@xsawyerx xsawyerx deleted the pr/fixupload branch September 27, 2016 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants