6

我为 SDL Tridion 2011 SP1 编写了一个 GUI 扩展。GUI 包含一个额外的功能区按钮和在保存组件时触发的事件处理程序。

我的事件处理程序注册如下：

PowerTools.Commands.ItemCommenting.prototype._execute = function (selection) {
    var item = $display.getItem();
    $evt.addEventHandler(item, "save", this.getDelegate(this._onItemSaved));
    $cme.getCommand("SaveClose")._execute(selection);
};

事件处理程序如下所示：

PowerTools.Commands.ItemCommenting.prototype._onItemSaved = function (eventitem) {

    var comment = prompt("Please enter a comment", "");

    $messages.registerNotification("Saving user comments...");

    var commentitemid = eventitem.source.getId();
    var commenterid = eventitem.source.getCreatorId();
    var commenter = $tcm.getItem(commenterid);
    var commentername = commenter.getDescription();
    var commentdate = eventitem.source.getLastModifiedDate();
    var commentversion = eventitem.source.getVersion();

    //Call the service to update 
    PowerTools.Model.Services.AppDataServices.Append("ext:ItemCommenting", commentitemid, "<comment><user>" + commenterid + "</user><message>" + comment + "</message><datetime>" + commentdate + "</datetime><version>" + commentversion + "</version></comment>", null, null, null, false);

};

这工作正常，除了变量commentername始终未定义。是否有更好的方法来获取用户的名称和描述？

此外，有谁知道返回的值eventitem.source.getCreatorId()实际上是 Reviser 还是创建该项目的人？

4

2 回答 2

8

我通常在安圭拉遵循这种方法：

  1. 用于$models.getItem(item Id)加载对象，从某人（@puf？）那里听说这是缓存的。
  2. 检查 object.isLoaded() 是否存在，如果是，则执行我的事件处理程序
  3. 如果对象未加载，则监听事件

这一切都归结为这样的事情：

p.keyword = $models.getItem(p.keywordUri);
if (p.keyword.isLoaded()) {
    this._onReleaseKeywordLoaded();
} else {
    $evt.addEventHandler(p.keyword, "load", this.getDelegate(this._onReleaseKeywordLoaded));
    p.keyword.load();
}

然后，您将从事件处理程序中调用模型的 Web 服务，因为您确定届时将加载该对象。

在您当前的代码中，您可能试图在加载对象之前阅读描述，因此未定义。我倾向于将我需要的变量放在this.propertiesvar 中的多个函数中（p在我的示例中），然后在每个函数的开头执行类似的操作：

var p = this.properties;
var whatever = p.whatever;
于 2012-10-09T18:13:32.227 回答
1

正如 Nuno 所说，您可能正在读取尚未加载的属性。

但是，无论如何，您都不应该接受客户端发送的名称。您应该在 Web 服务中获取当前用户的 ID 和名称。日期/时间等其他一些参数也是如此。

基本上，您的 Web 服务不应该信任传入的数据。因此，它可以自行发现的任何内容甚至都不应该成为该方法的参数，并且它确实需要的任何内容都应该被视为恶意内容并在您使用它之前进行清理。

于 2012-10-10T10:05:45.650 回答