Opened 4 years ago

Closed 4 years ago

#10365 closed defect (fixed)

Can not save pictures from the olpc library with Browse to Journal

Reported by: erikos Owned by: godiard
Priority: normal Milestone: 10.1.3
Component: browse-activity Version: 1.5/1.0 Software Build os852 aka 10.1.2
Keywords: Cc: godiard
Blocked By: Blocking:
Deployments affected: Action Needed: no action
Verified: no

Description

Steps to reproduce:

  • open Browse and go to the images category
  • view an image
  • right click and use the option 'download image'. Nothing happens.

The code that handles this is here: http://git.sugarlabs.org/projects/browse/repos/mainline/blobs/master/palettes.py#line224

Attachments (1)

0001-fix-10365.-Only-backport-solution-from-Browse-in-Sug.patch (1.3 KB) - added by godiard 4 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 4 years ago by godiard

  • Action Needed changed from diagnose to test in build

comment:2 Changed 4 years ago by erikos

Great, I new I have seen that issue already - thanks for the patch.

comment:3 follow-up: Changed 4 years ago by martin.langhoff

Is the patch broken for other uri schemes like https, file and ftp?

(Bonus question: what's the point of downloading a file: resource? ;-) -- However in Sugar it's neat "user" trick to get something into the Journal)

comment:4 follow-up: Changed 4 years ago by Quozl

  • Action Needed changed from test in build to package
  • Milestone changed from Not Triaged to 10.1.3
  • Version changed from Development build as of this date to 1.5/1.0 Software Build os852 aka 10.1.2

Tested patch on 10.1.2, can save pictures from the OLPC Library with Browse, the usual download sequence occurs.

Next step: package into Browse.

comment:5 Changed 4 years ago by godiard

  • Cc godiard added

comment:6 in reply to: ↑ 3 Changed 4 years ago by erikos

Replying to martin.langhoff:

Is the patch broken for other uri schemes like https, file and ftp?

No, they work fine. We just do check if the scheme is http before we manipulate the channel's referrer https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIHttpChannel#Attributes

(Bonus question: what's the point of downloading a file: resource? ;-) -- However in Sugar it's neat "user" trick to get something into the Journal)

Right, it is another way of getting something into the Journal :) Btw, the patch does as well fix the FTP case, so it is good to get it in.

comment:7 Changed 4 years ago by erikos

  • Owner changed from erikos to godiard

Moving this one to Gonzalo, as he takes care of Browse (0.84) now.

comment:8 Changed 4 years ago by erikos

Oh, forgot a quick comment about the patch. Please Mark the bug in the comment SL #NUMBER so we know that we refer to the Sugar Labs tracker here.

comment:9 in reply to: ↑ 4 Changed 4 years ago by erikos

Replying to Quozl:

Tested patch on 10.1.2, can save pictures from the OLPC Library with Browse, the usual download sequence occurs.

Thanks for testing. I did test the FTP case as working now fine with this patch.

Next step: package into Browse.

Gonzalo will take care of it.

comment:10 Changed 4 years ago by erikos

This one is ready to be pushed to the 0.84 branch.

comment:11 Changed 4 years ago by godiard

Pushed to git

comment:12 Changed 4 years ago by erikos

  • Action Needed changed from package to test in build

Will be in 353.

comment:13 Changed 4 years ago by greenfeld

  • Action Needed changed from test in build to no action
  • Resolution set to fixed
  • Status changed from new to closed

I was able to save images to the Journal from both Browse as well as from HTTP-based websites in 10.1.3 os353.

Note: See TracTickets for help on using tickets.