AnantLabs / codesmith

Automatically exported from code.google.com/p/codesmith
1 stars 0 forks source link

PLINQO v4.0:LinqCacheDataSource issue #241

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Revision 1191 has a problem with e.Result changed to source OnSelected
event, please fix:

-            Context.Cache.Insert(key, e.Result, null,
-                                 DateTime.Now.AddSeconds(Duration), 
-                                
System.Web.Caching.Cache.NoSlidingExpiration);
+            provider.Set(key, source);

should be:

Original issue reported on code.google.com by kachalkov on 2 Feb 2010 at 8:06

GoogleCodeExporter commented 9 years ago
Can you provide a specific code snippet that re-creates the issue.

Also, it looks like the description of the issue was cut off.  There does not 
appear to 
be anything after "should be:"

Original comment by shannon....@gmail.com on 2 Feb 2010 at 4:39

GoogleCodeExporter commented 9 years ago
http://code.google.com/p/codesmith/source/diff?spec=svn1191&r=1191&format=side&p
ath=/trunk/Templates/Frameworks/PLINQO/Source/CodeSmith.Data/Web/LinqCacheDataSo
urce.cs

In line 90 it should cache e.Result rather than source, see the attached patch 
from
latest version.

Original comment by kachalkov on 2 Feb 2010 at 10:57

Attachments:

GoogleCodeExporter commented 9 years ago
patch applied, thank you.

Original comment by paul.wel...@gmail.com on 1 Mar 2010 at 3:02

GoogleCodeExporter commented 9 years ago

Original comment by paul.wel...@gmail.com on 10 Mar 2010 at 11:27

GoogleCodeExporter commented 9 years ago
Can you please also check another issue with this control.
Method GetKey will return the same key for the WhereParameters like this:

<WhereParameters>
  <asp:QueryStringParameter Name="ApplicationId" QueryStringField="Id" Type="String" />
</WhereParameters>

So, this is a big problem then, any ideas how to store query by the SQL query 
with parameters? 

Original comment by kachalkov on 31 Aug 2010 at 6:56

GoogleCodeExporter commented 9 years ago

Original comment by paul.wel...@gmail.com on 31 Aug 2010 at 9:25