True that. I will make the change for consistency


On Mon, May 13, 2013 at 2:43 PM, Ji Park <notifications@github.com> wrote:

> In src/com/example/budgetmanager/AddEntryActivity.java:
>
> > -			Toast.makeText(this, "Please specify an amount.",
> > -					Toast.LENGTH_LONG).show();
> > -		} else {
> > -			// create the Entry object to add to the Budget
> > -			final Entry newEntry = createEntry();
> > -
> > -			if (newEntry == null) {
> > -				// do nothing until add Budget activity is up
> > -				return;
> > -			}
> > +		
> > +		if (mAmountView.getText().toString().isEmpty()) {
> > +			mAmountView.setError(getString(R.string.error_invalid_amount));
> > +			mAmountView.requestFocus();
> > +			return;
> > +		}
> >
>
> Couldn't find it because it wasn't in the strings associated with
> addentryactivity. Maybe we should factor the ones out of the ones that are
> being used in multiple activities like this one. Seems inconsistent that
> the error message being used in addentryactivity is located in the strings
> for add budget.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/coldstar96/cse403/pull/42/files#r4203413>
> .
>