Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 436045 - iOS toolkit - new action "UITabBar - check badge value"
Summary: iOS toolkit - new action "UITabBar - check badge value"
Status: CLOSED FIXED
Alias: None
Product: Jubula
Classification: Technology
Component: RC (show other bugs)
Version: unspecified   Edit
Hardware: All Mac OS X
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Kiss Tamás CLA
QA Contact: Oliver Goetz CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 437162
  Show dependency tree
 
Reported: 2014-05-28 08:11 EDT by Kiss Tamás CLA
Modified: 2014-08-05 09:18 EDT (History)
2 users (show)

See Also:


Attachments
Badge value patch file (4.33 KB, patch)
2014-05-28 08:11 EDT, Kiss Tamás CLA
markus.tiede: iplog+
Details | Diff
Badge value patch file (1.94 KB, patch)
2014-07-02 08:53 EDT, Kiss Tamás CLA
no flags Details | Diff
Badge value patch file (2.42 KB, patch)
2014-07-02 10:15 EDT, Kiss Tamás CLA
markus.tiede: iplog+
Details | Diff
Remove default value from operator param (1.09 KB, patch)
2014-07-07 09:23 EDT, Kiss Tamás CLA
no flags Details | Diff
Test results for check badge value - work as expected (9.92 KB, application/zip)
2014-07-09 03:22 EDT, Maissmaallsmyss Maulhs-Vvuillss CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kiss Tamás CLA 2014-05-28 08:11:34 EDT
Created attachment 243623 [details]
Badge value patch file

add check badge value method to component xml and I18nStrings patch file.
Comment 1 Kiss Tamás CLA 2014-05-28 09:08:33 EDT
This contribution complies with http://www.eclipse.org/legal/CoO.php
Comment 2 Kiss Tamás CLA 2014-05-28 09:11:35 EDT
Comment on attachment 243623 [details]
Badge value patch file

This contribution complies with http://www.eclipse.org/legal/CoO.php
Comment 4 Markus Tiede CLA 2014-06-24 03:28:52 EDT
We've just noticed that the action lacks of an operator for the badge values string comparison - @TKi: Could you please fix this and provide a small patch to address this?
Comment 5 Maissmaallsmyss Maulhs-Vvuillss CLA 2014-06-26 03:09:43 EDT
As discussed with TK, badge value can only be a number and though it is implemented as a string on Apple´s side, we decided to have an integer parameter in Jubula not to confuse users.
Comment 6 Kiss Tamás CLA 2014-06-27 05:11:03 EDT
I just checked the badge value and I realised the Integer value is won't be good, because this badge value can any string. That's why I will remain string and will add the operator parameter.
Comment 7 Kiss Tamás CLA 2014-07-02 08:53:46 EDT
Created attachment 244733 [details]
Badge value patch file

- Change the name to "Check Badge Value".
- Add operator to check badge value string.

This contribution complies with http://www.eclipse.org/legal/CoO.php
Comment 8 Markus Tiede CLA 2014-07-02 10:05:59 EDT
This patch (attachment 244733 [details]) has not been created from a commit - please do so so that we can maintain the correct author information. Thanks!
Comment 9 Kiss Tamás CLA 2014-07-02 10:15:24 EDT
Created attachment 244735 [details]
Badge value patch file

Patch created from commit

- Change the name to "Check Badge Value".
- Add operator to check badge value string.

This contribution complies with http://www.eclipse.org/legal/CoO.php
Comment 12 Kiss Tamás CLA 2014-07-07 09:23:44 EDT
Created attachment 244854 [details]
Remove default value from operator param

Provide patch file to remove default value from operator parameter


This contribution complies with http://www.eclipse.org/legal/CoO.php
Comment 13 Maissmaallsmyss Maulhs-Vvuillss CLA 2014-07-08 03:20:31 EDT
Tamas, thanks for contributions! I´ve noticed that in case of not existing badge we have an action error with description "action error". Maybe it would be better to have some more detailed description, e.g. "Badge value does not exist", similar to a description that you have made for wrong index "Action Error: invalid index"? Everything else looks good.
Comment 14 Kiss Tamás CLA 2014-07-09 03:14:22 EDT
Change the error message when there is no badge value. And I also changed the actual value and expected value

https://bitbucket.org/jubula/com.bredexsw.jubula.core/commits/474fcc41ca5539f288763aad1a879e083393b123
Comment 15 Maissmaallsmyss Maulhs-Vvuillss CLA 2014-07-09 03:22:16 EDT
Created attachment 244911 [details]
Test results for check badge value - work as expected

I´ve tested that now for no present badge we have correct description and also Pattern and Actual values are in right fields of Properties View.
Comment 16 Oliver Goetz CLA 2014-08-05 09:18:36 EDT
Closed due to comments 14 and 15