815063 - [CALL LOG] Outgoing calls are not appearing in call log

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Dialer, defect, P1)

Description

[Carlos Martínez Toral [:carlosmartinez]]

Tested in unagi device with:

Gaia - 56e45e8
Gecko - 4eaacd3

STR:
1-Place a call to another phone number

Expected result --> An outgoing call is registered in the call log
Actual result --> Nothing is registered in the call log

Comment 1

[David Scravaglieri [:scravag]]

related to https://bugzilla.mozilla.org/show_bug.cgi?id=814573

Comment 2

[diana.garcia]

Tested in unagi device with:

Gaia - 176aea9
Gecko - 4eaacd3

STR:
1-Place a call to another phone number

Expected result --> An outgoing call is registered in the call log
Actual result --> Only the first call is registered in the call log

Comment 3

[Etienne Segonzac (:etienne)]

The call log regressed apparently:

E/GeckoConsole(  489): [JavaScript Error: "TypeError: target.dataset.num is undefined" {file: "app://communicatio
ns.gaiamobile.org/dialer/js/recents.js" line: 432}]

Comment 4

[Etienne Segonzac (:etienne)]

Talked to Francisco and apparently German is the man for this bug :)

Comment 5

[Germán Toro del Valle (:gtorodelvalle)]

As commented with :oteo , the bug is solve so I am evolving it to RESOLVED ;-)

Comment 6

[rafael.marquez]

In today's version the calls which are answered by the other device, are not shown in the call log.

Gecko-04c2d4f
Gaia-1087df8

Comment 7

[rafael.marquez]

The bug is reproduced with the Telefonica build and the build of mozilla (unagi Gaia nightly on beta)

Comment 8

[Etienne Segonzac (:etienne)]

Talked with Vivien, stealing the bug :)

Comment 9

[Dylan Oliver [:doliver]]

Smoketest blocker -- raising priority.

Comment 10

[Etienne Segonzac (:etienne)]

Attachment: Pointer to gaia PR 6733

Comment 11

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 686805 [details]
Pointer to gaia PR 6733

Great!

Code looking good to me, also passed the unit tests and in the phone works like charm.

Thanks Etienne!

Comment 12

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/30192267eb0ddc05bacce217086709a1b35b5645

Comment 13

[mlevin]

Verified fixed on Unagi build 20130103070201.