Beatriz Rodríguez [:brg] Reporter
	Description • 9 years ago

Testing in master and 2.2, it is not possible to share contact info from contact detail panel while it is working very welll from Export option inside Contacts app.

STR:
- Contacts -> Open a Contact info panel -> Share Contact --> Bluetooth
- Pair with another device Flame or android (not possible to test with iphone due to bug 1144604).
- Flames try to share the vcard and starts BT transfer and then two scenarios had been detected:
A.- Nothing is received in the other device. See attachment: BT transfer starts but nothing else happens
B.- A corrupted file is shared with the other side which cannot be opened. Find attach the three screenshots of the process with a lot of unknown labels.

Please keep in mind that using the same devices the option to Export contacts via BT is working very well without the unknow labels and being able to receive the contact info.

Environmental variables:
Today's (3/18) v2.2 build:
Flame device
Gecko: 3e33c08
Gaia: 306772a
Build Id: 20150318081247
Platform version: 37.0

Today's (3/18) master build:
Flame device
Gecko: cebcc63
Gaia: 70bac3a
Build Id: 20150318082350
Plarform version: 39.0a1

	Beatriz Rodríguez [:brg] Reporter
	Comment 2 • 9 years ago

This is a broken feature tested during certification process.

	Johan Lorenzo [:jlorenzo]
	Comment 3 • 9 years ago

Comms triage: Broken new functionality.

	Maria Angeles Oteo (:oteo)
	Comment 4 • 9 years ago

Adrian, please, can you have a look at it? 
Tracing I've seen this error, that seems to be related to some issue between Contact and BT activities:

W/Bluetooth Manager(10586): [JavaScript Error: "TypeError: activity.source.data.filepaths is undefined" {file: "app://bluetooth.gaiamobile.org/js/transfer.js" line: 206}]

Thanks a lot!

	Adrián de la Rosa Assignee
	Comment 5 • 9 years ago

Taking it, I'll give it a look!

	Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)
	Comment 6 • 9 years ago

bug 1140247 should fix this problem. Can you verify the latest version again?

	Maria Angeles Oteo (:oteo)
	Comment 7 • 9 years ago

(In reply to Shawn Huang [:shawnjohnjr] from comment #6)
> bug 1140247 should fix this problem. Can you verify the latest version again?

We reported this issue after landing bug 1140247 in master branch and as you can see in comments 32, 33 and 36 of that bug 1140247, it explains:
"There is a new problem here,The contact is shared and can be received by the other device.  However the received file is considered unknown and cannot be opened" , 

That's the fault reported by Beatriz in this bug (scenario B) and it continues happening and is different from the one reported in bug 1140247.

	Beatriz Rodríguez [:brg] Reporter
	Comment 8 • 9 years ago

Already clarified by comment 7. Thanks Maria!

	Adrián de la Rosa Assignee
	Comment 10 • 9 years ago

Comment on attachment 8583104 [details] [review]
[gaia] ADLR-es:1144610 > mozilla-b2g:master

Simpler fix than I thought. The bluetooth activity was expecting a File object just because it requires a filename to be able to send the file. I just created a File object from the vCard blob and everything works as expected.

	Francisco Jordano [:arcturus] [:francisco]
	Comment 11 • 9 years ago

Comment on attachment 8583104 [details] [review]
[gaia] ADLR-es:1144610 > mozilla-b2g:master

Stealing the review, tried on the phone and working for me.

	GitHub Autolander Bot
	Comment 12 • 9 years ago

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/5c546a319e22e1e0b20608cd82069a8aeb05d5ca

	Francisco Jordano [:arcturus] [:francisco]
	Comment 13 • 9 years ago

Adrian, could you ask for uplift?

	Francisco Jordano [:arcturus] [:francisco]
	Comment 14 • 9 years ago

Comment on attachment 8583104 [details] [review]
[gaia] ADLR-es:1144610 > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined:
[Testing completed]:
Tested on device
[Risk to taking this patch] (and alternatives if risky):
Low
[String changes made]:

	Ryan VanderMeulen [:RyanVM]
	Comment 15 • 9 years ago

v2.2: https://github.com/mozilla-b2g/gaia/commit/f46ef0bff222694d36e4f9a5437bbb2a978c5eff

	Beatriz Rodríguez [:brg] Reporter
	Comment 16 • 9 years ago

Tested with todays builds in Flame master and v2.2, it is working well, I can share contact info from Contact details panel and I have not seen any "unknown" label during the process. Thanks for fixing it.

Master build details: Gecko-3e6e4cf.Gaia-4bb3a93
v2.2 build: Gecko-2b3fa8b.Gaia-8b3086a

	Shally Li（leave from Mozilla）
	Comment 17 • 9 years ago

According to the STR of Comment 0, this bug has been successfully verified on latest Nightly Flame v2.2&3.0. 

Actual results:Can share contact info via bluetooth from contact detail panel,and other devices(Flame or Android) can receive/open VCF file normally.

See attachment: verified_v2.2&3.0.MP4
Reproduce rate: 0/5

----------------------------------------------------------------------------
Device: Flame 2.2 build(Pass)
Build ID               20150409002503
Gaia Revision          ea735c21bfb0d78333213ff0376fce1eac89ead6
Gaia Date              2015-04-07 20:58:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/0efd5cdbe224
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150409.041814
Firmware Date          Thu Apr  9 04:18:23 EDT 2015
Bootloader             L1TC000118D0

Device: Flame 3.0 build(Pass)
Build ID               20150409160204
Gaia Revision          eaf69e651e19b98c096f8e63b9829fb31df50927
Gaia Date              2015-04-09 12:43:21
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/dd32e3ff3717
Gecko Version          40.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150409.193255
Firmware Date          Thu Apr  9 19:33:06 EDT 2015
Bootloader             L1TC000118D0