Review Board 1.7.22


Rave 70 - Add widget tag

Review Request #3437 - Created Jan. 9, 2012 and updated

jennifer lin
Reviewers
rave
rave
Rave 70 - add widget tags

 
Review request changed
Updated (Jan. 9, 2012, 10:02 p.m.)
Ship it!
Posted (Jan. 11, 2012, 6:12 p.m.)
Overall this looks good but I will be making the following changes while applying the patch:

1) UI tweaks to widget.jsp and store.jsp due to merge conflicts
2) Copy/paste text errors in DefaultWidgetTagPermissionEvaluator
3) WidgetApi#getTags: removed the "if" code branch where the widgetId is not > 0 and returns getAllTags().  Instead, I created a separate web service end point to call getAllTags()