Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Fix the order of some Android X checks to avoid conflicts with Support v4 #2169

Merged
WonderCsabo merged 5 commits into from
Oct 13, 2018
Merged

Fix the order of some Android X checks to avoid conflicts with Support v4 #2169

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
02577d3
Only check for support v4 Fragment if AndroidX Fragment is missing
dodgex Oct 13, 2018
0922a08
Check for Android X ActivityCompat first
dodgex Oct 13, 2018
3a6518f
Check for Android X ContextCompat before checking for support v4 version
dodgex Oct 13, 2018
3adc034
Check for Android X OnPreferenceClickListener first
dodgex Oct 13, 2018
508479f
Check for Android X OnPreferenceChangeListener first
dodgex Oct 13, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Check for Android X ContextCompat before checking for support v4 version
  • Loading branch information
@dodgex
dodgex committed Oct 13, 2018
commit 3a6518f65c35ff621e8477fa4f729986041676cc
6 changes: 3 additions & 3 deletions .../main/java/org/androidannotations/internal/core/handler/ContextCompatAwareResHandler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,10 @@ abstract class ContextCompatAwareResHandler extends AbstractResHandler {

@Override
protected IJExpression getInstanceInvocation(EComponentHolder holder, JFieldRef idRef, IJAssignmentTarget fieldRef, JBlock targetBlock) {
if (hasTargetMethodInContextCompat()) {
return getClasses().CONTEXT_COMPAT.staticInvoke(androidRes.getResourceMethodName()).arg(holder.getContextRef()).arg(idRef);
} else if (hasTargetMethodInAndroidxContextCompat()) {
if (hasTargetMethodInAndroidxContextCompat()) {
return getClasses().ANDROIDX_CONTEXT_COMPAT.staticInvoke(androidRes.getResourceMethodName()).arg(holder.getContextRef()).arg(idRef);
} else if (hasTargetMethodInContextCompat()) {
return getClasses().CONTEXT_COMPAT.staticInvoke(androidRes.getResourceMethodName()).arg(holder.getContextRef()).arg(idRef);
} else if (shouldUseContextMethod()) {
return holder.getContextRef().invoke(androidRes.getResourceMethodName()).arg(idRef);
} else if (!shouldUseContextMethod() && hasTargetMethodInContext()) {
Expand Down