Karl/BobStore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

checkstyle: Add MemberName

Browse Source
This commit is contained in:
Daniel Martí 2015-10-09 10:55:27 +02:00
parent 48b7096523
commit b5f62c03cb
4 changed files with 25 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
F-Droid/src/org/fdroid/fdroid/AppDetails.java
View File

@ -287,11 +287,11 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
holder.nativecode.setVisibility(View.GONE); holder.nativecode.setVisibility(View.GONE);
} }
if (apk.incompatible_reasons != null) { if (apk.incompatibleReasons != null) {
holder.incompatibleReasons.setText( holder.incompatibleReasons.setText(
getResources().getString( getResources().getString(
R.string.requires_features, R.string.requires_features,
apk.incompatible_reasons.toPrettyString())); apk.incompatibleReasons.toPrettyString()));
holder.incompatibleReasons.setVisibility(View.VISIBLE); holder.incompatibleReasons.setVisibility(View.VISIBLE);
} else { } else {
holder.incompatibleReasons.setVisibility(View.GONE); holder.incompatibleReasons.setVisibility(View.GONE);
@ -1062,7 +1062,7 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
private static boolean view_all_description; private static boolean view_all_description;
private static LinearLayout ll_view_more_description; private static LinearLayout ll_view_more_description;
private static LinearLayout ll_view_more_permissions; private static LinearLayout ll_view_more_permissions;
private final View.OnClickListener expander_permissions = new View.OnClickListener() { private final View.OnClickListener expanderPermissions = new View.OnClickListener() {
@Override @Override
public void onClick(View v) { public void onClick(View v) {
final TextView permissionListView = (TextView) ll_view_more_permissions.findViewById(R.id.permissions_list); final TextView permissionListView = (TextView) ll_view_more_permissions.findViewById(R.id.permissions_list);
@ -1077,7 +1077,7 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
} }
} }
}; };
private ViewGroup layout_links; private ViewGroup layoutLinks;
public AppDetailsSummaryFragment() { public AppDetailsSummaryFragment() {
prefs = Preferences.get(); prefs = Preferences.get();
@ -1133,18 +1133,18 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
return s.subSequence(0, i + 1); return s.subSequence(0, i + 1);
} }
private ViewGroup layout_links_content; private ViewGroup layoutLinksContent;
private View.OnClickListener expander_links = new View.OnClickListener() { private View.OnClickListener expanderLinks = new View.OnClickListener() {
@Override @Override
public void onClick(View v) { public void onClick(View v) {
TextView linksHeader = (TextView) layout_links.findViewById(R.id.information); TextView linksHeader = (TextView) layoutLinks.findViewById(R.id.information);
if (layout_links_content.getVisibility() == View.GONE) { if (layoutLinksContent.getVisibility() == View.GONE) {
layout_links_content.setVisibility(View.VISIBLE); layoutLinksContent.setVisibility(View.VISIBLE);
linksHeader.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(getActivity(), R.drawable.ic_website), null, ContextCompat.getDrawable(getActivity(), R.drawable.ic_expand_less_grey600), null); linksHeader.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(getActivity(), R.drawable.ic_website), null, ContextCompat.getDrawable(getActivity(), R.drawable.ic_expand_less_grey600), null);
} else { } else {
layout_links_content.setVisibility(View.GONE); layoutLinksContent.setVisibility(View.GONE);
linksHeader.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(getActivity(), R.drawable.ic_website), null, ContextCompat.getDrawable(getActivity(), R.drawable.ic_expand_more_grey600), null); linksHeader.setCompoundDrawablesWithIntrinsicBounds(ContextCompat.getDrawable(getActivity(), R.drawable.ic_website), null, ContextCompat.getDrawable(getActivity(), R.drawable.ic_expand_more_grey600), null);
} }
} }
@ -1185,7 +1185,7 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
} }
}; };
private final View.OnClickListener expander_description = new View.OnClickListener() { private final View.OnClickListener expanderDescription = new View.OnClickListener() {
public void onClick(View v) { public void onClick(View v) {
final TextView description = (TextView) ll_view_more_description.findViewById(R.id.description); final TextView description = (TextView) ll_view_more_description.findViewById(R.id.description);
final TextView view_more_permissions = (TextView) ll_view_more_description.findViewById(R.id.view_more_description); final TextView view_more_permissions = (TextView) ll_view_more_description.findViewById(R.id.view_more_description);
@ -1215,13 +1215,13 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
if (description.getLineCount() > MAX_LINES) { if (description.getLineCount() > MAX_LINES) {
description.setMaxLines(MAX_LINES); description.setMaxLines(MAX_LINES);
description.setEllipsize(TextUtils.TruncateAt.MARQUEE); description.setEllipsize(TextUtils.TruncateAt.MARQUEE);
description.setOnClickListener(expander_description); description.setOnClickListener(expanderDescription);
view_all_description = true; view_all_description = true;
ll_view_more_description = (LinearLayout) view.findViewById(R.id.ll_description); ll_view_more_description = (LinearLayout) view.findViewById(R.id.ll_description);
ll_view_more_description.setOnClickListener(expander_description); ll_view_more_description.setOnClickListener(expanderDescription);
view_more_description.setOnClickListener(expander_description); view_more_description.setOnClickListener(expanderDescription);
} else { } else {
view_more_description.setVisibility(View.GONE); view_more_description.setVisibility(View.GONE);
} }
@ -1240,11 +1240,11 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
summaryView.setText(getApp().summary); summaryView.setText(getApp().summary);
layout_links = (ViewGroup) view.findViewById(R.id.ll_information); layoutLinks = (ViewGroup) view.findViewById(R.id.ll_information);
layout_links_content = (ViewGroup) layout_links.findViewById(R.id.ll_information_content); layoutLinksContent = (ViewGroup) layoutLinks.findViewById(R.id.ll_information_content);
final TextView linksHeader = (TextView) view.findViewById(R.id.information); final TextView linksHeader = (TextView) view.findViewById(R.id.information);
linksHeader.setOnClickListener(expander_links); linksHeader.setOnClickListener(expanderLinks);
// Website button // Website button
View tv = view.findViewById(R.id.website); View tv = view.findViewById(R.id.website);
@ -1326,7 +1326,7 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
if (!getApks().isEmpty() && (curApkCompatible || prefs.showIncompatibleVersions())) { if (!getApks().isEmpty() && (curApkCompatible || prefs.showIncompatibleVersions())) {
// build and set the string once // build and set the string once
buildPermissionInfo(); buildPermissionInfo();
permissionHeader.setOnClickListener(expander_permissions); permissionHeader.setOnClickListener(expanderPermissions);
} else { } else {
permissionHeader.setVisibility(View.GONE); permissionHeader.setVisibility(View.GONE);
@ -1643,7 +1643,7 @@ public class AppDetails extends AppCompatActivity implements ProgressListener, A
public static class AppDetailsListFragment extends ListFragment { public static class AppDetailsListFragment extends ListFragment {
private final String SUMMARY_TAG = "summary"; private static final String SUMMARY_TAG = "summary";
private AppDetailsData data; private AppDetailsData data;
private AppInstallListener installListener; private AppInstallListener installListener;

4
F-Droid/src/org/fdroid/fdroid/UpdateService.java
View File

@ -482,10 +482,10 @@ public class UpdateService extends IntentService implements ProgressListener {
final List<String> reasons = checker.getIncompatibleReasons(apk); final List<String> reasons = checker.getIncompatibleReasons(apk);
if (reasons.size() > 0) { if (reasons.size() > 0) {
apk.compatible = false; apk.compatible = false;
apk.incompatible_reasons = Utils.CommaSeparatedList.make(reasons); apk.incompatibleReasons = Utils.CommaSeparatedList.make(reasons);
} else { } else {
apk.compatible = true; apk.compatible = true;
apk.incompatible_reasons = null; apk.incompatibleReasons = null;
} }
} }
} }

6
F-Droid/src/org/fdroid/fdroid/data/Apk.java
View File

@ -42,7 +42,7 @@ public class Apk extends ValueObject implements Comparable<Apk> {
public int repoVersion; public int repoVersion;
public String repoAddress; public String repoAddress;
public Utils.CommaSeparatedList incompatible_reasons; public Utils.CommaSeparatedList incompatibleReasons;
public Apk() { public Apk() {
size = 0; size = 0;
@ -94,7 +94,7 @@ public class Apk extends ValueObject implements Comparable<Apk> {
nativecode = Utils.CommaSeparatedList.make(cursor.getString(i)); nativecode = Utils.CommaSeparatedList.make(cursor.getString(i));
break; break;
case ApkProvider.DataColumns.INCOMPATIBLE_REASONS: case ApkProvider.DataColumns.INCOMPATIBLE_REASONS:
incompatible_reasons = Utils.CommaSeparatedList.make(cursor.getString(i)); incompatibleReasons = Utils.CommaSeparatedList.make(cursor.getString(i));
break; break;
case ApkProvider.DataColumns.REPO_ID: case ApkProvider.DataColumns.REPO_ID:
repo = cursor.getInt(i); repo = cursor.getInt(i);
@ -147,7 +147,7 @@ public class Apk extends ValueObject implements Comparable<Apk> {
values.put(ApkProvider.DataColumns.PERMISSIONS, Utils.CommaSeparatedList.str(permissions)); values.put(ApkProvider.DataColumns.PERMISSIONS, Utils.CommaSeparatedList.str(permissions));
values.put(ApkProvider.DataColumns.FEATURES, Utils.CommaSeparatedList.str(features)); values.put(ApkProvider.DataColumns.FEATURES, Utils.CommaSeparatedList.str(features));
values.put(ApkProvider.DataColumns.NATIVE_CODE, Utils.CommaSeparatedList.str(nativecode)); values.put(ApkProvider.DataColumns.NATIVE_CODE, Utils.CommaSeparatedList.str(nativecode));
values.put(ApkProvider.DataColumns.INCOMPATIBLE_REASONS, Utils.CommaSeparatedList.str(incompatible_reasons)); values.put(ApkProvider.DataColumns.INCOMPATIBLE_REASONS, Utils.CommaSeparatedList.str(incompatibleReasons));
values.put(ApkProvider.DataColumns.IS_COMPATIBLE, compatible ? 1 : 0); values.put(ApkProvider.DataColumns.IS_COMPATIBLE, compatible ? 1 : 0);
return values; return values;
} }

2
config/checkstyle/checkstyle.xml
View File

@ -19,7 +19,7 @@
<!--<module name="ConstantName" />--> <!--<module name="ConstantName" />-->
<!--<module name="LocalFinalVariableName" />--> <!--<module name="LocalFinalVariableName" />-->
<!--<module name="LocalVariableName" />--> <!--<module name="LocalVariableName" />-->
<!--<module name="MemberName" />--> <module name="MemberName" />
<module name="MethodName" /> <module name="MethodName" />
<module name="PackageName" /> <module name="PackageName" />
<module name="ParameterName" /> <module name="ParameterName" />