views:

152

answers:

3

I have the following code but I am getting the following compile errors:

Attribute 'WebPartStorage' is not valid on this declaration type. It is only valid on 'property, indexer' declarations.

AND

Attribute 'FriendlyName' is not valid on this declaration type. It is only valid on 'property, indexer' declarations.

I have modified my code from the MSDN article: http://msdn.microsoft.com/en-us/library/dd584174(office.11).aspx. Does anyone have any idea what I am doing wrong that is causing this error?

  [Category("Custom Properties")]
    [DefaultValue(RegionEnum.None)]
    [WebPartStorage(Storage.Shared)]
    [FriendlyName("Region")]
    [Description("Select a value from the dropdown list.")]
    [Browsable(true)]
    protected RegionEnum _Region;
    public RegionEnum Region
    {
        get
        {
            return _Region;
        }
        set
        {
            _Region = value;
        }
    }
+1  A: 

The message tells you, doesn't it? You are trying to set the attribute to a field, but it is only valid on indexers and properties.

protected RegionEnum _Region;

[Category("Custom Properties")]
[DefaultValue(RegionEnum.None)]
[Description("Select a value from the dropdown list.")]
[Browsable(true)]
[WebPartStorage(Storage.Shared)]
[FriendlyName("Region")]
public RegionEnum Region
{
    get
    {
        return _Region;
    }
    set
    {
        _Region = value;
    }
}
Maximilian Mayerl
+3  A: 

You seem to have attached the attribute to the field; attributes always adhere to the next thing (in this case, the field). You should re-order so that they adhere to the property instead of the field.

BTW; protected fields are rarely a good idea (they should be private); but especially if the property is public: what is the point?

protected RegionEnum _Region;
[Category("Custom Properties")]
[DefaultValue(RegionEnum.None)]
[WebPartStorage(Storage.Shared)]
[FriendlyName("Region")]
[Description("Select a value from the dropdown list.")]
[Browsable(true)]
public RegionEnum Region
{
    get { return _Region; }
    set { _Region = value; }
}
Marc Gravell
A: 

Hopefully you have using Microsoft.SharePoint.WebPartPages;, have you?

Janis Veinbergs