Sometimes that name gets updated. It'd be cool to have this.

CommentFileSizeAuthor
Screen Shot 2012-10-04 at 7.51.27 PM.png64.39 KBdiwant

Comments

diwant’s picture

diwant commented
Assigned: diwant » Unassigned

If someone can point me to where that message is coming from, I'll do it.

merlinofchaos’s picture

merlinofchaos commented

This message might be coming from CTools' export UI, though I thought it was better about using administrative names. The first place to look would be in plugins/export_ui/ in mini panels but chances are it's just inheriting from the base class in ctools/plugins/export_ui

diwant’s picture

diwant commented

Thanks Earl! I found the part in CTools export plugin ui where it has the 'Blah blah has been updated' message, but that uses %title. I think that means that somewhere we are passing the wrong argument in for title to the plugin. I'm going to take a deeper look.

$plugin['strings']['confirmation']['edit'] += array(
    'success' => t('%title has been updated.'),
    'fail' => t('%title could not be updated.'),
  );

(that's line 347 in export-ui.inc)

diwant’s picture

diwant commented
Assigned: Unassigned » diwant
diwant’s picture

diwant commented

CTools grabs the field it uses in the title from $this->plugin['export']['key']. I'm looking at who sets that up now.

diwant’s picture

diwant commented

@merlinofchaos this seems like it's a CTools decision to use name instead of title. You're on ctools as well. Should I raise an issue there and see if people want that change? Or should I upload a patch to fix it locally here in Minipanels?

merlinofchaos’s picture

merlinofchaos commented

Yeah, this should be moved to CTools and patched there so everything can use it.

diwant’s picture

diwant commented
Status: Active » Closed (duplicate)

Reported at CTools (1805982)