The following document contains the results of FindBugs Report
FindBugs Version is 1.3.8
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 466 | 10 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method call in com.atlassian.gadgets.opensocial.internal.ShindigActivityServiceImpl.createActivity(UserId, GroupId, String, Set, Activity, SecurityToken) passes null to a nonnull parameter of com.atlassian.gadgets.opensocial.spi.PersonService.getPerson(String, Set, SecurityToken) | CORRECTNESS | NP_NONNULL_PARAM_VIOLATION | 69 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should com.atlassian.gadgets.renderer.internal.AtlassianContainerConfig$Containers be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 28 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class com.atlassian.gadgets.renderer.internal.BlobCrypterImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 40-51 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class com.atlassian.gadgets.renderer.internal.GadgetSpecFactoryImpl$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 89-123 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in com.atlassian.gadgets.renderer.internal.GadgetSpecUrlCheckerImpl.assertRenderable(String) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 89-92 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class com.atlassian.gadgets.renderer.internal.guice.ShindigModule$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 98 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class com.atlassian.gadgets.renderer.internal.guice.ShindigModule$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 99 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 42 | Low |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| HTTP parameter written to Servlet error page in com.atlassian.gadgets.renderer.internal.servlet.GadgetSpecUrlRenderPermissionServletFilter.doFilter(ServletRequest, ServletResponse, FilterChain) | SECURITY | XSS_REQUEST_PARAMETER_TO_SEND_ERROR | 86 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.atlassian.gadgets.renderer.internal.servlet.RpcRelay is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 16-34 | Low |
