Automatically ignore static fields

[aemeryjsa]

Currently, static fields in the model contribute to the model definition.

For example, the following class contains the static, final field MAX_ROOMS.

public class House extends RealmObject {
    private int roomCount;
    public static final int MAX_ROOMS = 10;
}

Attempting to compile results in the the following compile-time exception:
Error:The fields of the model must be private

I don't think static fields should contribute to the model definition. They belong to the class, not the instance, so it doesn't make sense for them to become a model property. They should be implicitly ignored.

Adding the Ignore annotation to the field allows it to compile:

public class House extends RealmObject {
    private int roomCount;
    @Ignore public static final int MAX_ROOMS = 10;
}

Also, the error message could be improved by mentioning what field in what class caused the exception.

Realm: 0.70.1
Android Studio: 0.8.11
OS: Window 8

[emanuelez]

Good suggestion! We will look into it.
Realm 0.71.0 comes with a more verbose annotation processor. Check it out!
:)

On Wed, Oct 8, 2014 at 3:44 PM, aemeryjsa notifications@github.com wrote:

Currently, static fields in the model contribute to the model definition.

For example, the following class contains the static, final field
MAX_ROOMS.

public class House extends RealmObject {
private int roomCount;
public static final int MAX_ROOMS = 10;}

Attempting to compile results in the the following compile-time exception:
Error:The fields of the model must be private

I don't think static fields should contribute to the model definition.
They belong to the class, not the instance, so it doesn't make sense for
them to become a model property. They should be implicitly ignored.

Adding the Ignore annotation to the field allows it to compile:

public class House extends RealmObject {
private int roomCount;
@ignore public static final int MAX_ROOMS = 10;}

Also, the error message could be improved by mentioning what field in what
class caused the exception.

Realm: 0.70.1
Android Studio: 0.8.11
OS: Window 8

—
Reply to this email directly or view it on GitHub
#464.

[cmelchior]

Should be fixed with this pull request: #538.

[Groxx]

fwiw that pull request appears to only ignore static methods. I haven't read it closely enough to be certain, but the comments imply it, and at the very least Realm 0.76.0 doesn't ignore my static fields.

[denley]

This is not fixed. I am having the same issue using 0.76.0.

Static methods are ignored properly, but not static fields.

[mpost]

Confirmed. public static final fields are not ignored in 0.78.0.

[freezy]

Also it crashes with the dreaded NPE that doesn't give any indication at all what the problem might be.

[mpost]

The debug message leaves something to be desired indeed. When running code without an @Ignore from inside AndroidStudio it doesn't hint at what the problem is. Only when running a gradlew assemble -info do we get proper error messages.