inline edit support not working

Question

I have this code with inline edit support. When I hit save the changes are not saved - I see that the currentCatalogItem which is sent to be updated does not have the value I changed.

What am I doing wrong?

Page:

<apex:commandButton action="{!saveCatalogItem}" id="saveButton" value="Save" style="display:none"/>    

<apex:pageBlockSection columns="2">
   <apex:outputField value="{!Portfolio_Catalog_Item__c.edit_Short_Name__c}">
         <apex:inlineEditSupport showOnEdit="saveButton, cancelButton" event="ondblclick" hideOnEdit="editButton, approveButton, deleteButton, cloneButton" changedStyleClass="myBoldClass" resetFunction="resetInlineEdit"/>
    </apex:outputField>
<apex:pageBlockSection columns="2">

Controller Extension:

public pageReference saveCatalogItem() {
       try {
           update currentCatalogItem;
       }
       catch(Exception e){
           ApexPages.addmessage(new ApexPages.message(ApexPages.severity.Error, e.getMessage()));
       }
       return null;
}

Answer 1

I have also faced the same issue. You can resolve this by first changing the focus of the Inline Edit to some other element (say a hidden input type). This will send your updated value back to the controller.

Try the below updated code.

Page:

<script>
function callMyFunc(){
   document.getElementById('focusMe').click();
   actionFuncToUpdate();    
 }
</script>
<apex:actionFunction action="{!saveCatalogItem}" name="actionFuncToUpdate"/>
<input type="text" id="focusMe" style="display:none" />
<apex:commandButton onClick="callMyFunc();return false;" id="saveButton" value="Save" style="display:none"/>    

    <apex:pageBlockSection columns="2">
       <apex:outputField value="{!Portfolio_Catalog_Item__c.edit_Short_Name__c}">
             <apex:inlineEditSupport showOnEdit="saveButton, cancelButton" event="ondblclick" hideOnEdit="editButton, approveButton, deleteButton, cloneButton" changedStyleClass="myBoldClass" resetFunction="resetInlineEdit"/>
        </apex:outputField>
    <apex:pageBlockSection columns="2">

Controller Extension:

public pageReference saveCatalogItem() {
       try {
           update currentCatalogItem;
       }
       catch(Exception e){
           ApexPages.addmessage(new ApexPages.message(ApexPages.severity.Error, e.getMessage()));
       }
       return null;
}

Hope this helps.

Answer 2

The standard object below needs to be referenced in your save method -> or simply don't override the standard save method since you're using standardController and not doing anything special in your save method

    <apex:outputField value="{!Portfolio_Catalog_Item__c.edit_Short_Name__c}" />

Also - you could do this:

<apex:commandButton action="{!saveCatalogItem}" id="saveButton" value="Save" style="display: none" />
<apex:inlineEditSupport showOnEdit="saveButton, cancelButton" event="ondblclick" hideOnEdit="editButton, approveButton, deleteButton, cloneButton" changedStyleClass="myBoldClass" resetFunction="resetInlineEdit"/>
<apex:pageBlockSection columns="2">
    <apex:outputField value="{!catalogItem.edit_Short_Name__c}" />
</apex:pageBlockSection>

Controller:

public class MyController{

    public Portfolio_Catalog_Item__c catalogItem { get; set; }

    public MyController( ApexPages.standardController controller ){
          catalogItem = ( Portfolio_Catalog_Item__c )controller.getRecord();
    }

    public PageReference saveCatalogItem(){
        try{
            update catalogItem;
         }
         catch( Exception ex ){
             ApexPages.addMessage( new ApexPages.Message( ApexPages.Severity.ERROR, ex.getMessage() ));
         }

        return null;
    }

}

if you have a need to update this one item only and not the entire page - i can see the necessity to override the standard save method while doing no logic - as you would not be saving the entire page - however, given your example i don't really see that. simplest fix is just this:

I am also assuming that you're updating the rendered part of this button in JS code

<apex:commandButton value="Save" action="{!save}" style="display: none" />