Mic Bowman
5a2895977a
Make JsonStore path parsing more robust. Should fix the
...
invalid path problem.
2013-02-20 23:28:46 +00:00
Oren Hurvitz
7fc820b3fd
Fixed ReadSculptData(): the check whether there are enough bytes to read was incorrect
2013-02-20 23:22:29 +00:00
Oren Hurvitz
72838a04d6
Fixed check for 0 results in GetUserAccounts()
2013-02-20 23:22:22 +00:00
Mic Bowman
9ee6c06ec8
Broaden the internal OSD type checks to parse JSON that has
...
non string values.
2013-02-20 23:20:33 +00:00
Justin Clark-Casey (justincc)
5393ecfa75
Don't allow exceptions to propogate from FlotsamAssetCache which may occur when deleting expired files or stamping the region status file.
...
Changes various error level log lines to warn since these are not fatal to the operation of OpenSimulator
2013-02-20 23:10:49 +00:00
Mic Bowman
f6ddd20413
Fix the return values for JsonDestroyStore, JsonRemoveValue, and JsonSetValue.
...
Fix the link message status when reading a notecard.
2013-02-20 23:10:42 +00:00
Justin Clark-Casey (justincc)
b46a9cf57f
If a component of a coalesced object fails to deserialization, do not add a null where the object should be.
...
This prevents a later load IAR failure.
This code is currently only used by IAR loading.
2013-02-20 23:10:34 +00:00
Justin Clark-Casey (justincc)
aab719dc18
On IAR loading, if loading of a coaleseced item entirely fails, then continue with the IAR load rather than failing completely.
2013-02-20 23:10:28 +00:00
Justin Clark-Casey (justincc)
90d71d423c
Refine TestJsonReadNotecard() and use / instead of . to separate paths.
...
An attack of the stupid meant that I was using / as a path separator.
Fixing this makes the tests behave better, though still with some questions.
Also, I imagine / shouldn't really put data in the root as that's not a valid identifier.
This commit also fix the / mistake in other tests those this does not affect their outcomes.
2013-02-20 23:10:22 +00:00
Justin Clark-Casey (justincc)
3f6feec914
Extend TestJsonReadNotecard() for reads to non-root locations and fake stores.
...
Assertions for loading to non-root paths are currently commented out because this doesn't seem to be working. Will be raising mantis to resolve.
2013-02-20 23:10:15 +00:00
Justin Clark-Casey (justincc)
6874d56452
Stop wrongly create a receiving store already populated with "Hello":"World" in TestJsonReadNotecard()
2013-02-20 23:10:08 +00:00
Justin Clark-Casey (justincc)
23df5768c3
Extend TestJsonWriteNotecard() regression test for cases with fake paths and fake stores.
...
Also separates out TestJsonWriteReadNotecard() into separate write and read tests
2013-02-20 23:10:03 +00:00
Justin Clark-Casey (justincc)
f52cee9732
Extend TestJsonSetValue() regressio ntes tto test against a fake store
2013-02-20 23:09:58 +00:00
Justin Clark-Casey (justincc)
37e4186ad8
Extend TestJsonTestPath() regression test with checks against fake values and fake stores
2013-02-20 23:09:54 +00:00
Justin Clark-Casey (justincc)
5be2483e93
Add tests for removing fake values/from fake store in TestJsonRemoveValue()
...
Again, need to check if returning true for removing a value that doesn't exist is most appropriate.
2013-02-20 23:09:49 +00:00
Justin Clark-Casey (justincc)
6418b89fd6
Add testing for getting non-existing values and values from a non-existing datastore to TestJsonGetValue()
2013-02-20 23:09:44 +00:00
Justin Clark-Casey (justincc)
c67f791097
Add TestJsonDestoreStoreNotExists()
...
This still returns true even if we ask to destroy a store that does not exist.
Need to check that this is more appropriate behaviour.
2013-02-20 23:09:36 +00:00
Justin Clark-Casey (justincc)
001bbb0b16
Fix a recent regression in e17392a
where JsonSetValue() stopped working (probably other functions as well).
...
Fix is to call through to the no-arg constructor from the string constructor in JsonStore, which I suspect was just forgotten.
This was actually picked up by the TestJsonSetValue() regression test failing
But this isn't being run on jenkins due to the .net version issue.
This commit also puts the full stack trace in logged messages and makes these error level messages instead of info
2013-02-20 23:08:46 +00:00
Justin Clark-Casey (justincc)
365292e38f
Add TestJsonWriteReadNotecard() regression test
2013-02-20 23:00:08 +00:00
Justin Clark-Casey (justincc)
fb2de29f77
Make json store tests operate on a single thread to ensure we don't run into any race related test failures in the future.
2013-02-20 22:59:38 +00:00
Dan Lake
251a70338a
WebStats will now use actual logfile as specified in OpenSim.exe.config rather than hardcoded ./OpenSim.log. This allows for rotating logs and other file appender types
2013-02-20 22:59:31 +00:00
Justin Clark-Casey (justincc)
e210dc188a
Remove very long unused IScriptHost and NullScriptHost
2013-02-20 22:58:48 +00:00
Justin Clark-Casey (justincc)
e61117669e
Remove unused ScriptEngineLoader and ScriptEngineInterface in OpenSim.Region.Framework.dll
...
I believe this predates the generic system of registering interfaces and is very long unused.
2013-02-20 22:58:41 +00:00
Justin Clark-Casey (justincc)
97fcc97902
Remove wrong code in ScriptUtils.GetAssetIdFromKeyOrItemName which would return the item ID instead of the asset ID if the identifier was a uuid that matched an inventory item name.
...
This would practically never happen.
This makes this overloaded version of the function consistent with the other version.
It looks like this accidentally came over in commit c5af16a
from Tue Oct 16 12:40:21 2012
However, there's arguably a case for looking for an item name that matches a UUID before assuming that the identifier is already an asset ID.
2013-02-20 22:58:37 +00:00
Justin Clark-Casey (justincc)
eab1b1b9f8
Allow JsonReadNotecard() to accept the name of the notecard as well as the asset ID.
...
Agreed in discussion with cmickeyb.
This is to make this consistent with similar existing LSL/OSSL functions such as llTriggerSound() and osNpcLoadAppearance() that allow an item name or an asset id.
2013-02-20 22:58:31 +00:00
Justin Clark-Casey (justincc)
87d50974f8
refactor: Move functions that lookup asset ids from task inventory or pass them through to ScriptUtils class in OpenSim.Region.Framework.dll
...
Renames functions to better reflect what they do.
This is so that code registering with modInvoke() can reuse this code to provide functions that behave in a consistent manner with existing LSL/OSSL functions.
2013-02-20 22:57:33 +00:00
Justin Clark-Casey (justincc)
78814adf01
Update ScriptModuleComms and interface wholesale from 0.7.5 as of commit 99ebff94
...
The number of deltas are making this too hard to update through cherry-picking
2013-02-20 22:43:24 +00:00
SignpostMarv
341dcbede6
moving code that will be common into private static method
2013-02-20 22:36:36 +00:00
teravus
901aa4153e
* the root prim was being given an OffsetPosition in addition to setting the position when creating the root prim. The offset position caused the positioning code to re-move the root prim when you selected it and released it.
2013-02-20 22:27:17 +00:00
Melanie
f99ba6f506
Try to fix uploaded mesh rotations - code from Avination code base.
2013-02-20 22:27:00 +00:00
Justin Clark-Casey (justincc)
85f32f184c
Fix bug where viewers would not see the "Module command functions not enabled" error if these were disabled and a viewer attempted to call one.
...
This was not working because the shouter was wrongly signalled as an agent rather than a prim
2013-02-20 22:26:53 +00:00
Justin Clark-Casey (justincc)
982d3d3faa
minor: change comment about assuming script references are textures in UuidGatherer since this is no longer the behaviour.
2013-02-20 22:26:15 +00:00
Justin Clark-Casey (justincc)
744ed1b313
Log missing assets on "fcache assets" found. This ignores references found by scanning LSL/notecard files since these are the source of false positives.
...
This also changes UuidGatherer to reutn an AssetType.Unknown for embedded script/lsl references instead of Texture, since these are often not textures.
This is added to help people in determining when they have missing assets such as textures, etc.
In this case, one wants to run "fcache clear" first.
2013-02-20 22:26:04 +00:00
Mic Bowman
69ca1498ef
Move the JsonStore regular expressions to static variables to avoid
...
recompiling on every operation. Added JsonList2Path script function
to simplify array iteration.
2013-02-20 22:25:47 +00:00
Mic Bowman
9a60039d36
Add JsonTestStore to determine if a JsonStore is associated with
...
a particular UUID.
2013-02-20 22:25:34 +00:00
Justin Clark-Casey (justincc)
f030f1dbfb
Add JsonDestroyStore() basic regression test
2013-02-20 22:25:18 +00:00
Justin Clark-Casey (justincc)
16641e5bd0
refactor: Make invocations of json store functions from the regression test simpler
2013-02-20 22:25:10 +00:00
Justin Clark-Casey (justincc)
10db078aa4
Add regression test for script func JsonRemoveValue()
2013-02-20 22:25:04 +00:00
Justin Clark-Casey (justincc)
afc93d7ac4
minor: change name of json script tests to JsonStoreScriptModuleTests instead of copy/pasted LSL_ApiInventoryTests
2013-02-20 22:24:50 +00:00
Justin Clark-Casey (justincc)
fa5e1dde64
minor: remove EnableLogging() left over in CoopTerminationTests
2013-02-20 22:24:30 +00:00
Justin Clark-Casey (justincc)
0810f5f449
Try ignoring json tests if they can't be run due to being on <=.net 3.5
2013-02-20 22:24:25 +00:00
Justin Clark-Casey (justincc)
56e245e5e9
minor: Tidy up disabled logging on AssetTransactionModule for future use. Make it clear that transaction parameter to HandleUDPUploadRequest is an ID.
2013-02-20 22:24:20 +00:00
Justin Clark-Casey (justincc)
89362dbc4e
minor: remove some mono compile warnings in XEngine.cs
2013-02-20 22:24:13 +00:00
Melanie
2f6ddc68b0
Prevent items being destroyed by rename operations. Renaming of a wearable also
...
sends an asset transaciton but it is empty. So we can't ignore name data
when a transaction is present and can't treat every transaction as valid.
Conflicts:
OpenSim/Region/Framework/Scenes/Scene.Inventory.cs
2013-02-20 22:23:54 +00:00
Justin Clark-Casey (justincc)
25b539e593
Fix issue where the "set terrain texture" console command did not tell the viewers that textures had updated (hence they did not display the changes).
...
Addresses http://opensimulator.org/mantis/view.php?id=6513
2013-02-20 22:23:46 +00:00
Talun
25a9ad1722
Mantis 6343: Turn a prim to flexy to OFF don't work llSetPrimParams
...
Correction so that scripts can turn Flexi off as well as on.
2013-02-20 22:23:30 +00:00
Justin Clark-Casey (justincc)
d3bbfcc97b
minor: Fix full scene part console report to show proper Light* names rather than all wrongly FlexiDrag
2013-02-20 22:22:32 +00:00
Justin Clark-Casey (justincc)
c3cbba7778
minor: Call down to base OpenSimTestCase.SetUp() in NPCModuleTests to disable any enabled logging from previous tests
2013-02-20 22:22:26 +00:00
Justin Clark-Casey (justincc)
648932458a
Disable json tests for now - I see this is failing on jenkins because mono 2.4.3 doesn't have Funcs >5 params, though mono 2.6 onwards does.
...
Standardize logging in JsonStoreScriptModule and some minor code formatting.
2013-02-20 22:21:50 +00:00
Justin Clark-Casey (justincc)
f1b978cdf0
See if getting the registering JsonReadNotecard with MethodInfo works around an apparent issue with registering methods with more than 5 parameters on mono 2.4.3
2013-02-20 22:21:44 +00:00