Adding capability to specify primary key

[nickdjones]

This update allows the user to override the auto-generated primary key with one of their own. Useful for when using Sugar to synchronise data with another source.

Usage:

public class Book extends SugarRecord<Book> {
    public String name;

    public Book(String name) {
        this.name = name;
    }

    // Extra constructor to set id is not a requirement
    // User can continue to use Sugar without changing code
    public Book(Long id, String name) {
        this.id = id;
        this.name = name;
    }

Example:

Book autoBook = new Book("This book will get a generated ID like normal")
autoBook.save()
Book saved : 1

Book manualBook = new Book(99L, "This book has its ID specified manually")
manualBook.save()
Book saved : 99

[whoshuu]

I'd like to get some version of this in the codebase. Can you explain:

1. Why you are adding to the .gitignore file?
2. Why you make id public instead of protected?

[whoshuu]

I'd like to keep style changes separate from feature additions.

[nickdjones]

I added a .gitignore file because IDEs create a build folder that needs excluding.

You're right it should be protected rather than public.

[whoshuu]

Which IDE are you using? I'm using Eclipse and it doesn't create any of those files you're adding to the .gitignore.

[nickdjones]

Android Studio, which is superseding Eclipse as the recommended IDE for android development. I don't see how adding 2 lines to the ignore file is doing any harm?

[whoshuu]

Gotcha. Thanks for the heads up.

On Jul 4, 2014, at 7:30 AM, ItsNotWorking notifications@github.com wrote:

Android Studio, which is superseding Eclipse as the recommended IDE for android development.

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