diff options
Diffstat (limited to 'OpenSim')
-rw-r--r-- | OpenSim/Data/MySQL/MySQLAssetData.cs | 8 | ||||
-rw-r--r-- | OpenSim/Data/MySQL/MySQLXAssetData.cs | 10 | ||||
-rw-r--r-- | OpenSim/Data/PGSQL/PGSQLAssetData.cs | 10 | ||||
-rw-r--r-- | OpenSim/Data/SQLite/SQLiteAssetData.cs | 10 | ||||
-rw-r--r-- | OpenSim/Framework/AssetBase.cs | 16 |
5 files changed, 17 insertions, 37 deletions
diff --git a/OpenSim/Data/MySQL/MySQLAssetData.cs b/OpenSim/Data/MySQL/MySQLAssetData.cs index 83e7e4c..1488e1a 100644 --- a/OpenSim/Data/MySQL/MySQLAssetData.cs +++ b/OpenSim/Data/MySQL/MySQLAssetData.cs | |||
@@ -170,14 +170,6 @@ namespace OpenSim.Data.MySQL | |||
170 | } | 170 | } |
171 | 171 | ||
172 | string assetDescription = asset.Description; | 172 | string assetDescription = asset.Description; |
173 | if(assetDescription.Length > AssetBase.MAX_LMASSET_DESC) | ||
174 | { | ||
175 | if(asset.Type == (sbyte) AssetType.Landmark) | ||
176 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_LMASSET_DESC); | ||
177 | else | ||
178 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_ASSET_DESC); | ||
179 | } | ||
180 | |||
181 | if (asset.Description.Length > AssetBase.MAX_ASSET_DESC) | 173 | if (asset.Description.Length > AssetBase.MAX_ASSET_DESC) |
182 | { | 174 | { |
183 | assetDescription = asset.Description.Substring(0, AssetBase.MAX_ASSET_DESC); | 175 | assetDescription = asset.Description.Substring(0, AssetBase.MAX_ASSET_DESC); |
diff --git a/OpenSim/Data/MySQL/MySQLXAssetData.cs b/OpenSim/Data/MySQL/MySQLXAssetData.cs index 6c6f560..ec2bcc6 100644 --- a/OpenSim/Data/MySQL/MySQLXAssetData.cs +++ b/OpenSim/Data/MySQL/MySQLXAssetData.cs | |||
@@ -214,12 +214,12 @@ namespace OpenSim.Data.MySQL | |||
214 | } | 214 | } |
215 | 215 | ||
216 | string assetDescription = asset.Description; | 216 | string assetDescription = asset.Description; |
217 | if(assetDescription.Length > AssetBase.MAX_LMASSET_DESC) | 217 | if (asset.Description.Length > AssetBase.MAX_ASSET_DESC) |
218 | { | 218 | { |
219 | if(asset.Type == (sbyte) AssetType.Landmark) | 219 | assetDescription = asset.Description.Substring(0, AssetBase.MAX_ASSET_DESC); |
220 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_LMASSET_DESC); | 220 | m_log.WarnFormat( |
221 | else | 221 | "[XASSET DB]: Description '{0}' for asset {1} truncated from {2} to {3} characters on add", |
222 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_ASSET_DESC); | 222 | asset.Description, asset.ID, asset.Description.Length, assetDescription.Length); |
223 | } | 223 | } |
224 | 224 | ||
225 | if (m_enableCompression) | 225 | if (m_enableCompression) |
diff --git a/OpenSim/Data/PGSQL/PGSQLAssetData.cs b/OpenSim/Data/PGSQL/PGSQLAssetData.cs index 97ea6a4..81adb03 100644 --- a/OpenSim/Data/PGSQL/PGSQLAssetData.cs +++ b/OpenSim/Data/PGSQL/PGSQLAssetData.cs | |||
@@ -175,12 +175,12 @@ namespace OpenSim.Data.PGSQL | |||
175 | } | 175 | } |
176 | 176 | ||
177 | string assetDescription = asset.Description; | 177 | string assetDescription = asset.Description; |
178 | if(assetDescription.Length > AssetBase.MAX_LMASSET_DESC) | 178 | if (asset.Description.Length > AssetBase.MAX_ASSET_DESC) |
179 | { | 179 | { |
180 | if(asset.Type == (sbyte) AssetType.Landmark) | 180 | assetDescription = asset.Description.Substring(0, AssetBase.MAX_ASSET_DESC); |
181 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_LMASSET_DESC); | 181 | m_log.WarnFormat( |
182 | else | 182 | "[ASSET DB]: Description '{0}' for asset {1} truncated from {2} to {3} characters on add", |
183 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_ASSET_DESC); | 183 | asset.Description, asset.ID, asset.Description.Length, assetDescription.Length); |
184 | } | 184 | } |
185 | 185 | ||
186 | using (NpgsqlConnection conn = new NpgsqlConnection(m_connectionString)) | 186 | using (NpgsqlConnection conn = new NpgsqlConnection(m_connectionString)) |
diff --git a/OpenSim/Data/SQLite/SQLiteAssetData.cs b/OpenSim/Data/SQLite/SQLiteAssetData.cs index 9fbd9c7..9c2bd2e 100644 --- a/OpenSim/Data/SQLite/SQLiteAssetData.cs +++ b/OpenSim/Data/SQLite/SQLiteAssetData.cs | |||
@@ -143,12 +143,12 @@ namespace OpenSim.Data.SQLite | |||
143 | } | 143 | } |
144 | 144 | ||
145 | string assetDescription = asset.Description; | 145 | string assetDescription = asset.Description; |
146 | if(assetDescription.Length > AssetBase.MAX_LMASSET_DESC) | 146 | if (asset.Description.Length > AssetBase.MAX_ASSET_DESC) |
147 | { | 147 | { |
148 | if(asset.Type == (sbyte) AssetType.Landmark) | 148 | assetDescription = asset.Description.Substring(0, AssetBase.MAX_ASSET_DESC); |
149 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_LMASSET_DESC); | 149 | m_log.WarnFormat( |
150 | else | 150 | "[ASSET DB]: Description '{0}' for asset {1} truncated from {2} to {3} characters on add", |
151 | assetDescription = assetDescription.Substring(0, AssetBase.MAX_ASSET_DESC); | 151 | asset.Description, asset.ID, asset.Description.Length, assetDescription.Length); |
152 | } | 152 | } |
153 | 153 | ||
154 | //m_log.Info("[ASSET DB]: Creating Asset " + asset.FullID.ToString()); | 154 | //m_log.Info("[ASSET DB]: Creating Asset " + asset.FullID.ToString()); |
diff --git a/OpenSim/Framework/AssetBase.cs b/OpenSim/Framework/AssetBase.cs index 67239ec..87fd04a 100644 --- a/OpenSim/Framework/AssetBase.cs +++ b/OpenSim/Framework/AssetBase.cs | |||
@@ -51,8 +51,7 @@ namespace OpenSim.Framework | |||
51 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | 51 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); |
52 | 52 | ||
53 | public static readonly int MAX_ASSET_NAME = 64; | 53 | public static readonly int MAX_ASSET_NAME = 64; |
54 | public static readonly int MAX_ASSET_DESC = 127; | 54 | public static readonly int MAX_ASSET_DESC = 64; |
55 | public static readonly int MAX_LMASSET_DESC = 255; | ||
56 | 55 | ||
57 | /// <summary> | 56 | /// <summary> |
58 | /// Data of the Asset | 57 | /// Data of the Asset |
@@ -306,18 +305,7 @@ namespace OpenSim.Framework | |||
306 | public string Description | 305 | public string Description |
307 | { | 306 | { |
308 | get { return m_description; } | 307 | get { return m_description; } |
309 | set | 308 | set { m_description = value; } |
310 | { | ||
311 | if(value.Length > AssetBase.MAX_LMASSET_DESC) | ||
312 | { | ||
313 | if(m_type == (sbyte) AssetType.Landmark) | ||
314 | m_description = value.Substring(0, AssetBase.MAX_LMASSET_DESC); | ||
315 | else | ||
316 | m_description = value.Substring(0, AssetBase.MAX_ASSET_DESC); | ||
317 | } | ||
318 | else | ||
319 | m_description = value; | ||
320 | } | ||
321 | } | 309 | } |
322 | 310 | ||
323 | public DateTime CreationDate | 311 | public DateTime CreationDate |