Comments now fully working

Merge with caution. This one's gonna be dangerous.
No. Uuid.from string actually throws an npe.
On Nov 7, 2014 4:28 PM, "Stephen Just" <notifications@github.com> wrote:

> In
> QuestionApp/src/ca/ualberta/cs/cmput301f14t14/questionapp/view/ViewCommentDialogFragment.java:
>
> > @@ -25,20 +25,27 @@ public Dialog onCreateDialog(Bundle savedInstanceState) {
> >  		
> >  		LayoutInflater inflater = getActivity().getLayoutInflater();
> >  		
> > -		UUID questionId = UUID.fromString(getArguments().getString("questionId"));
> > -		UUID answerId = UUID.fromString(getArguments().getString("answerId"));
> > -		UUID commentId = UUID.fromString(getArguments().getString("commentId"));
> > +		//UUID.fromString NPEs if property is missing. Need to fix this.
> > +		UUID questionId = null;
> > +		UUID answerId = null;
> > +		UUID commentId = null;
> > +		
> > +		//These two should never be null/empty.
>
> You could just check (if .. != null) { ... }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CMPUT301F14T14/Project/pull/97/files#r20044867>.
>