Orientation fix missing in latest stable

RGJ · 3 בפברואר,‏ 2018,‏ 8:45am

Saturday morning is always a good moment to research weird problems.
I suddenly see things that I’ve overlooked for a long time.

The image orientation fix seems to be missing in the stable branch.

Can you please, please backport it for 1.9.3 ?

Master:

github.com/discourse/discourse

lib/upload_creator.rb

main


      
          def initialize(file, filename, opts = {})
            @file = file
            @filename = (filename || "").gsub(/[^[:print:]]/, "")
            @upload = Upload.new(original_filename: @filename, filesize: 0)
            @opts = opts
            @filesize = @opts[:filesize] if @opts[:external_upload_too_big]
            @opts[:validate] = (
              if opts[:skip_validations].present?

Stable:

github.com/discourse/discourse

lib/upload_creator.rb

stable


      
          def initialize(file, filename, opts = {})
            @file = file
            @filename = (filename || "").gsub(/[^[:print:]]/, "")
            @upload = Upload.new(original_filename: @filename, filesize: 0)
            @opts = opts
            @filesize = @opts[:filesize] if @opts[:external_upload_too_big]
            @opts[:validate] = (

codinghorror · 3 בפברואר,‏ 2018,‏ 3:16pm

Sure @zogstrip can you take a look?

RGJ · 4 בפברואר,‏ 2018,‏ 6:31pm

I did a diff between stable 1.9.0 and tests-passed 2.0 beta1, which should be pretty much the same, and I found three more differences which should not be there.

https://github.com/discourse/discourse/blob/stable/lib/validators/upload_url_validator.rb#L6
https://github.com/discourse/discourse/blob/master/lib/validators/upload_url_validator.rb#L6

https://github.com/discourse/discourse/blame/stable/lib/final_destination.rb#L208-L223
https://github.com/discourse/discourse/blame/v2.0.0.beta1/lib/final_destination.rb#L208-L218

https://github.com/discourse/discourse/blame/stable/app/assets/javascripts/discourse/templates/components/group-membership-button.hbs#L28-L31

RGJ · 16 בפברואר,‏ 2018,‏ 5:03pm

@zogstrip @neil do you have any plans to merge these changes back in to stable ?

zogstrip · 16 בפברואר,‏ 2018,‏ 5:06pm

Yup. Topic isn’t closed and still is assigned Just haven’t got a chance to do it yet.

RGJ · 17 במאי,‏ 2018,‏ 12:17pm

Would it be possible to merge the image rotation bugfix back into stable before Discourse 2.0 stable is being released? It’s only a single line in upload_creator.rb that is still causing a lot of headaches.

zogstrip · 17 במאי,‏ 2018,‏ 12:56pm

Sorry for the delay. It’s now backported

zogstrip · 17 במאי,‏ 2018,‏ 4:00pm

This topic was automatically closed after 3 hours. New replies are no longer allowed.

נושא		תגובות	צפיות
Orientation of images uploaded is wrong Support	19	2155	18 בינואר,‏ 2018
Is there a way to manually rotate an image Support	29	6648	12 בדצמבר,‏ 2020
Mobile image upload turns sideways Support	33	13226	23 ביוני,‏ 2017
Stable is well... broken Support	8	1134	5 באפריל,‏ 2018
2.8.2: Minor Patch Release Announcements release-notes	0	448	22 במרץ,‏ 2022

Orientation fix missing in latest stable

נושאים קשורים