[KARAF-3248] bundle:refresh doesn't work without argument

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.1
• Fix Version/s: 3.0.2, 4.0.0.M2, 4.0.0.M3
• Component/s: karaf
• Assignee: Jean-Baptiste Onofré
• Reporter: Jean-Baptiste Onofré
• Created: 24/Sep/14 19:22
• Updated: 15/Sep/18 16:35
• Resolved: 01/Oct/14 09:18

Description

When using bundle:refresh without argument:

karaf@root()> bundle:refresh

it should refresh all bundles. But actually, it doesn't do anything as the test is not correct (in the case of bundles is null):

wiring.refreshBundles(bundles == null || bundles.isEmpty() ? null : bundles);

It should be:

if (bundles == null || bundles.isEmpty()) {
  bundles = null;
}
wiring.refreshBundles(bundles);