Exposed methods for disabling/enabling platform menu items in the contex...

[rcruz]

...t menu

[rcruz]

API Docs: blackberry/WebWorks-API-Docs#403

[ejzn]

listDisabledPlatformItems doesn't seem to work. But the WP API works.

[rcruz]

Fixed listDisabledPlatformItems()

[ejzn]

@ishneur Can you test this one out?

[jeffheifetz]

Missing functional tests

[nukulb]

@ishneur can you give this a quick test

[ishneur]

Tested and works correctly with some issues/comments:

• Non-platform items like Share and Cancel items can be disabled and enabled.
• disablePlatformItem/enablePlatformItem for not supported action for a particular context returns "true", e.g. disable Save Image for Link
• For listDisabledPlatformItems(), items disabled for CONTEXT_ALL are listed only for CONTEXT_ALL and not for other contexts (even though they are disabled for other contexts too), but I don't know if they are supposed to be

[nukulb]

@erikj54 - is this expected?

[ejzn]

• Cancel is a platform action, so yes, we would like it disabled. I'm not sure what you mean by Share @ishneur?
• 2nd one is an issue, it should probabyl return a verbose error right @rcruz
• I think we should add the CONTEXT_ALL items to ever other context when it returns the disabled list. @rcruz can we do this too? It could be done both in WW or WP depending on timing.

[ishneur]

@erikj54 When I set actionId = "MenuService-0" inside disablePlatformItem(), the Share menu item is disabled.

[ishneur]

Another issue I noticed is that if you disable an item (e.g. ACTION_COPY) for CONTEXT_ALL, and then try to enable it for a particular context, e.g. CONTEXT_LINK, it doesn't work and returns "false", so the only way to enable it again is to enable it for CONTEXT_ALL.

The same issue happens the other way around: if you disable an item for CONTEXT_LINK and then try to enable it for CONTEXT_ALL, it returns "false" too and the item is not added back for link.

I don't know if the above is intended or not, but it looks like the same context must be specified for enable/disable in order for it to work, even when enabling/disabling for CONTEXT_ALL.

[ejzn]

The above seems like a bug. @rcruz

[rcruz]

We'll fix the list to display a more accurate listing, and the CONTEXT_ALL combinations.

[nukulb]

FYI

depends on http://github.rim.net/webworks/webplatform/pull/259

targetted at 10.2, will merge after we branch