mishrsud / mvc-mini-profiler

Automatically exported from code.google.com/p/mvc-mini-profiler
0 stars 0 forks source link

Do not pollute the System namespace. #11

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
There are some helper functions that are listed as being in the System 
namespace.

Since many System namespaces are included in many different files, this 
drastically increases the likelihood of a naming collision in even unrelated 
files.

Since there is already a MvcMiniProfiler namespace and since these are only 
useful when used in conjunction with the mini profiler, the helper methods 
should be moved to that namespace.

Original issue reported on code.google.com by Spoonful...@gmail.com on 9 Jun 2011 at 8:32

GoogleCodeExporter commented 8 years ago

Original comment by nrcraver on 20 Jun 2011 at 12:03

GoogleCodeExporter commented 8 years ago
Can I second this?  I just spent twenty minutes trying to figure out why  
System.Data.Linq.DataContextUtils didn't exist in the System.Data.Linq.dll.  

The DataContextUtils should class be in the MvcMiniProfiler.Data namespace 
someplace.

Original comment by brianb...@gmail.com on 23 Jun 2011 at 9:37

GoogleCodeExporter commented 8 years ago
just changed this, which is going to break some of you EF folks on upgrade. 

I can not afford to have these EF tickets in the queue for so long. Agree this 
makes more sense though config becomes slightly more involved

Original comment by sam.saff...@gmail.com on 15 Aug 2011 at 1:04

GoogleCodeExporter commented 8 years ago
So where is it now?

Original comment by brianb...@gmail.com on 26 Aug 2011 at 7:49