/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.jackrabbit.jcr2spi.operation;

import org.apache.jackrabbit.jcr2spi.state.PropertyState;

import javax.jcr.RepositoryException;
import javax.jcr.ValueFormatException;
import javax.jcr.AccessDeniedException;
import javax.jcr.ItemExistsException;
import javax.jcr.UnsupportedRepositoryOperationException;
import org.apache.jackrabbit.value.QValue;

import javax.jcr.version.VersionException;
import javax.jcr.lock.LockException;
import javax.jcr.nodetype.ConstraintViolationException;

/**
 * <code>SetPropertyValue</code>...
 */
public class SetPropertyValue extends AbstractOperation {

    private final PropertyState propertyState;
    private final QValue[] values;
    private final int propertyType;

    private SetPropertyValue(PropertyState propertyState, int propertyType, QValue[] values) {
        this.propertyState = propertyState;
        this.propertyType = propertyType;
        this.values = values;
        
        addAffectedItemState(propertyState);
    }

    //----------------------------------------------------------< Operation >---
    /**
     *
     * @param visitor
     */
    public void accept(OperationVisitor visitor) throws ValueFormatException, LockException, ConstraintViolationException, AccessDeniedException, ItemExistsException, UnsupportedRepositoryOperationException, VersionException, RepositoryException {
        visitor.visit(this);
    }

    //----------------------------------------< Access Operation Parameters >---
    public PropertyState getPropertyState() {
        return propertyState;
    }

    public int getPropertyType() {
        return propertyType;
    }

    public QValue[] getValues() {
        return values;
    }

    //------------------------------------------------------------< Factory >---
    public static Operation create(PropertyState propState, QValue[] iva,
                                   int valueType) {
        SetPropertyValue sv = new SetPropertyValue(propState, valueType, iva);
        return sv;
    }
}
@@ -1297,16 +1297,67 @@
 ception;
+%0Aimport java.util.List;%0Aimport java.util.ArrayList;
 %0A%0A/**%0A *
@@ -1783,24 +1783,16 @@
 values;%0A
-        
 %0A       
@@ -2695,19 +2695,23 @@
 Value%5B%5D 
-iva
+qValues
 ,%0A      
@@ -2752,24 +2752,339 @@
 alueType) %7B%0A
+        // compact array (purge null entries)%0A        List list = new ArrayList();%0A        for (int i = 0; i %3C qValues.length; i++) %7B%0A            if (qValues%5Bi%5D != null) %7B%0A                list.add(qValues%5Bi%5D);%0A            %7D%0A        %7D%0A        QValue%5B%5D cleanValues = (QValue%5B%5D) list.toArray(new QValue%5Blist.size()%5D);%0A
         SetP
@@ -3144,19 +3144,27 @@
 ueType, 
-iva
+cleanValues
 );%0A     

work in progress