Dsc 543

Added display and allow editing of copy record details as per dsc-543.  
Must be older code or older coder  :-)
I think the types were required up to java 7

shaun

From: Aaron Muller <notifications@github.com<mailto:notifications@github.com>>
Reply-To: nla/banjo <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, 11 August 2014 11:19 AM
To: nla/banjo <banjo@noreply.github.com<mailto:banjo@noreply.github.com>>
Subject: Re: [banjo] Dsc 543 (#633)


In src/banjo/services/DataEditService.java:

> +    }
> +
> +    /**
> +     * Build an Amber Copy given the ParameterMap from a Spring WebRequest.
> +     *
> +     * Iterates over the given parameters and attempts to set the Amber Copy's
> +     * vertex attribute to what was provided in the map. If the value in the map
> +     * is blank, it removes the attribute from the copy's vertex.
> +     *
> +     * TODO: Validation or manipulation for particular attributes.
> +     *
> +     */
> +    private void buildCopy(String action, Copy copy, Map<String, String[]> parameterMap) throws JsonParseException, JsonMappingException, IOException, ParseException {
> +
> +        List<String> aliases = new ArrayList<String>();
> +        Map<String, String> otherNumbers = new HashMap<String, String>();


119,120: Type parameters shouldn't be required?

—
Reply to this email directly or view it on GitHub<https://github.com/nla/banjo/pull/633/files#r16035031>.