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

HoloEverywhere support #776

Description

@webhive

HoloEveywhere no more depends from ActionBarScherlock libary. It uses ActionBarCompat instead.

So this code in ThirdPartyLibHelper.java is not right anymore

if (qName.startsWith("org.holoeverywhere")) {
  /*
   * HoloEverywhere depends on ABS and uses its own provided
   * activity classes
   */
    return true;
}

Unfortunately it generates ActionBarScherlock code even if ActionBarScherlock not used in project. At least in my project I got issue with @optionsmenu annotation.

I am not familiar with AndroidAnnotations library code so can't to fix myself and prepare pull request, but at least I can point to problematic place. Hope it can be useful.

Activity

  1. DayS commented on Oct 31, 2013

    @DayS
    Contributor

    What if someone still using old version of HoloEveywhere ? In this case we still have to generate ActionBarScherlock's related code.
    Maybe we could provide a way to exclude some library related code with annotation processor option. Dunno

  2. webhive commented on Oct 31, 2013

    @webhive
    Author

    As I wrote above I am not familiar with library internals so can't answer what to do. You are right - anyway someone using old versions so need some support for both cases. As quick solution may be remove automatic detection of holoeverywhere and display some message instead about what to do or link to webpage with detailed descirption?

  3. webhive commented on Oct 31, 2013

    @webhive
    Author

    Hmm - look - may be I am wrong - I am not a java guru
    But may be codel like this could be useful

    try {
        Class.forName("com.actionbarsherlock.app.SherlockActivity");
        return true;
    } catch(Exception e) {
        return false;
    }
    

    So we just checking if ActionBarScherlock classes are present in the classpath and in case of HoloEverywhere it mean ActionBarScherlock used (including ActionBarCompat together with ActionBarScherlock caused compile error so this is impossible to have both library in one project)

    Suppose such way we can unambiguously know about use ActionBarScherlock or not.

  4. DayS commented on Nov 7, 2013

    @DayS
    Contributor

    Yes, we could check if ABS is present in classpath (not using this code. we have a helper somewhere to do this :)).
    I'll work on this point tomorrow.

  5. ghost assigned on Nov 7, 2013
  6. DayS commented on Nov 8, 2013

    @DayS
    Contributor

    It should fixed and retro-compatible. Please re-open if not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions