IPC Pricing routine for Product Family Margin in Quotation is not working

Question

Hi,
I am working on the IPC Pricing Routine to calculate Item's Product Family Margin in the Quotation. 
Process followed:
Step 1:  Created a Group condition ZPFM for Product Family Margin and assigned to the pricing procedure. 
Step 2:  Created a Value Routine to determine ZPFM's Condition Rate and Condition Value.
Logic: 
          1.  Collect (Sum Of)  the Cost (ZSVC) and the Subtotal 3 of the items which are in same Material Group.
           2.  Assign the ZPFM Condition Value = Sum of Subtotal 3 - Cost.
Issue:  The condition value for ZPFM is calculating correct for all line items except the last item.  I am getting some junk value always to the last item. If I delete the last time, again next last item is giving wrong value.  The calculation value in the routine debugging log (SM53) is showing correct value.  However, assigning the wrong value.  No other routines have been assigned to this condition type in the pricing procedure. 
Routine Code:
package stanley.pricing.userexits.val;
import java.math.BigDecimal;
import com.sap.spe.pricing.customizing.PricingCustomizingConstants;
import com.sap.spe.pricing.transactiondata.PricingTransactiondataConstants;
import com.sap.spe.pricing.transactiondata.userexit.IPricingConditionUserExit;
import com.sap.spe.pricing.transactiondata.userexit.IPricingItemUserExit;
import com.sap.spe.base.logging.UserexitLogger;
import com.sap.spe.pricing.transactiondata.userexit.ValueFormulaAdapter;
//import com.sap.spe.pricing.transactiondata.userexit.IPricingCondition;
public class Z_CondValueRoutine730 extends  ValueFormulaAdapter{
   private static UserexitLogger uelogger =
          new UserexitLogger(Z_CondValueRoutine730.class);
  public BigDecimal overwriteConditionValue(IPricingItemUserExit pricingItem,
         IPricingConditionUserExit pricingCondition) {
  // Variable Declarations.
  BigDecimal conditionValueZCVC=PricingTransactiondataConstants.ZERO;
  BigDecimal conditionValueZCVCAccrual=PricingTransactiondataConstants.ZERO;
  BigDecimal subTotal3=PricingTransactiondataConstants.ZERO;
  BigDecimal subTotal3Accrual=PricingTransactiondataConstants.ZERO;
  if(pricingCondition.getConditionTypeName().equalsIgnoreCase("ZPFM")){
            IPricingItemUserExit[] prItems;
            prItems = pricingItem.getUserExitDocument().getUserExitItems();
            String materialGroup = pricingItem.getAttributeValue("MATL_GRP");
            uelogger.writeLogDebug("  Material Group  :"+materialGroup);
            for (int i=0;i<prItems.length;i++){
                 String materialGroup1 = prItems[i].getAttributeValue("MATL_GRP");
                 if (materialGroup1.equalsIgnoreCase(materialGroup)){
                      subTotal3 = prItems[i].getSubtotal(PricingCustomizingConstants.ConditionSubtotal.SUBTOTAL_3).getValue();
                      subTotal3Accrual = subTotal3Accrual.add(subTotal3);
                      IPricingConditionUserExit[] conditionsForCumulation = prItems[i].getUserExitConditions();
                      for (int j = 0; j < conditionsForCumulation.length; j++) {
                           if(conditionsForCumulation[j].getConditionTypeName()==null) {
                               continue;
                           else if ( conditionsForCumulation[j].getConditionTypeName().equalsIgnoreCase("ZCVC")){
                                conditionValueZCVC = conditionsForCumulation[j].getConditionValue().getValue();
                                conditionValueZCVCAccrual = conditionValueZCVCAccrual.add(conditionValueZCVC);
            BigDecimal conditionValueZPFM=PricingTransactiondataConstants.ZERO;
            conditionValueZPFM = subTotal3Accrual.subtract(conditionValueZCVCAccrual);
           BigDecimal conditionRateZPFM=PricingTransactiondataConstants.ZERO;
           conditionRateZPFM = conditionValueZPFM.multiply(new BigDecimal ("100"));
          conditionRateZPFM = conditionRateZPFM.divide(subTotal3Accrual,7,BigDecimal.ROUND_FLOOR);
         uelogger.writeLogDebug(subTotal3Accrual+"-"+conditionValueZCVCAccrual+" = "+conditionValueZPFM);
         uelogger.writeLogDebug("Condition Rate  :"+conditionRateZPFM);
       pricingCondition.setConditionRateValue(conditionRateZPFM.setScale(2,BigDecimal.ROUND_HALF_UP));
       pricingCondition.setConditionValue(conditionValueZPFM.setScale(2,BigDecimal.ROUND_HALF_UP));
          return null;
       return null;

Answer 1

Hi,
I think you should change xkwert variable instead of komv-kwert.
Standard routines does the same.
Aslo check in SPRO settings that routine is attached and
getting triggered by putting break-point.
Regards,
Vishal