delete button on subpanel row: why does this work and is it safe to use?

Hi,

I guess you already solve it. Just wanted to know if the answer was something like this,

({
  extendsFrom: 'SubpanelListView',
  initialize: function(options){
     this._super('initialize', [options]);
     this.context.on('list:drawereditrow:fire',this.drawerEditClicked, this);
     this.context.on('list:deleterow:fire',this.deleteClicked, this);
  },
  deleteClicked: function(model) {
     this._super('deleteModel');

  },  
  drawerEditClicked: function(model){
    app.drawer.open({
       layout:'create-actions',
       context:{
         create: true,
         model: model,
         module:'oppp_Opportunity_Products',
       }
    });
  },
  _dispose: function() {
    this._super('_dispose');
  },
});

Hello Francesca Shiekh,

I'm trying to do the same as you did, for a Sugar Sell V11.1 but the event is not fired, especially this part : 

deleteClicked: function (model) {
        this._super('deleteModel');
    },

When I check on the base subpanel-list.js file, there is no deleteModel function, it is located in recordlist.js (base) function.

Is there anything I'm missing ? 

Thanks a lot for your answer.

Best regards,

Enes

Finally I made it working by removing the content of the function deleteClicked ... very strange :/

Gosh, it's been a few years since this thread...

Here is what I have for my subpanel-list on that module today, and it's working appropriately for me on Sugar 11.0.3 Professional.

Notice the initialize function has the deletedrow:fire event, and I'm not using deleteModel but rather the deleteClicked function uses the original warnDelete(model) which I believe subpanel-list inherits from recordlist.js

({
  extendsFrom: 'SubpanelListView',
  initialize: function(options){
     this._super('initialize', [options]);
     this.context.on('list:drawereditrow:fire',this.drawerEditClicked, this);
     this.context.on('list:deleterow:fire',this.deleteClicked, this);
    if(this.layout){
      this.layout.on('list:record:deleted', function() {
        this.refreshCollection();
        this.refreshParentAmount();
      }, this);
    }
  },
  refreshParentAmount: function(){
    var parentmodel = this.context.parent.get('model');
    parentmodel.fetch({
      view: undefined,
      fields: ['amount', 'product_of_interest_c']
    });
  },
  deleteClicked: function(model) {
    var self = this;
    self.warnDelete(model);
  },
  drawerEditClicked: function(model){
    app.drawer.open({
       layout:'create',
       context:{
         create: true,
         model: model,
         module:'oppp_Opportunity_Products',
       }
    });
  },
});

Thanks a lot for your feedback Francesca, very nice from you

We will apply this piece of code on monday.

Have a nice week-end !

Enes

Hi Francesca,

It's ok on our side, the modification works on Sugar Ent 11.0.2 :-)

Thanks a lot and have a nice week.

Enes

Hi Francesca,

It's ok on our side, the modification works on Sugar Ent 11.0.2 :-)

Thanks a lot and have a nice week.

Enes