Comments by Matthew Dennis (Top 133 by date)

as ASD asd ASD

This is a test comment.

test

another test

testing 1,2,3

test comment

test comment

And I have my expected 7 :). I tweaked a few conditionals that might have resulted in some notification not being sent. I hope this works now :)

In order to map business requirements to BDD test I will be using SpedFlow. This is a free extension for VS. Each test project needs to load the Nuget SpecFlow package to get the required dll.
It looks like Selenium is the UI automation test tool of choice. I had thought to use WaitN, but it appears to no longer being actively developed.
A Nuget package, FluentAutomation supports both Selenium and WaitN, and simplifies writing the tests.

Customized the templates to use the Bootstrap styling consistently.

It just worked.

This corrects a number of issues. The rate of updates means this will be in final beta or release when we are ready for release.
Will use the Samples as a starting point.

Looks like an Issue with the UserPrefernces Caching

UserPreferences do not have the correct values.
May need some help on this one.

Confirmed that OriginalGriff had comments to Q&A on June 5th that do not have corresponding entries in the Notifications table.

Traced the code, and it seems to be working correctly.
Check members' Email Enables and Email confirmation dates that reported the issue and they seem correct.
This could mean that the UserPreference data is incorrect either in the cookie, session, Redis, or database. I don't think it is the email server as there are reports of Emails but no Notifications.
Need to check if there is a multi-threading issue.

need to check the maintainence downtimes for DFP API. I know that with DART, we couldn't use the API on Staurdays. This would prevent getting Ads.

Was a type in url.
However, the url should be HTTPS, not HTTP except in development.

Cached the UserAuthenticationToken in Request.Items so that the same token is used for the duration of the Request.

The problem is that the Url registered in the LinkedIn account does not match http://www.codeproject.com:80/script/Membership/OAuthLogon.aspx.

I don’t have access to the account to make the required change.

I have reassigned the task to Chris.

Currently (26May14) have created a SVN-Git repository containing ALL the projects. Need to break it apart into separate repositories.

Tested, but most uses of location info use the UserPreferences for this information.
Tidied up the campsite a bit while I was in there.

http://www.codeproject.com/Messages/4809551/Article-Points.aspx
http://www.codeproject.com/Messages/4804232/Article-published-after-edit-by-an-editor-will-not.aspx

Unable to recreate. May have been an issue due to lack of disk space, which has been resolved.

Not as simple as I thought it was going to be.

1. Need to Store filter in Campaign.

2. Status Indicators for both filtered and non-filtered contacts.

3. Need to be able to change filters at report.

4. Need to be able to export filtered and non-filtered versions.

Currently Article Edit Locks are maintained in the database in the Article table using the EditLockGained (DateTime?) and EditLockMemberId (int?) fields.

Unfortunately, the checking for the lock occurs in the code, not the database which can result in multiple users thinking they have the lock.
One of the reasons why the check is done in the code, is that the time stored is the time the lock was taken, not the time it should expire.

The current flow is

1. Is the article locked by someone else?
1. Set the article fields to Now and the EditorMemberId without checks.
1. Do stuff, possible over several Requests.
1. Release the lock, again without checks for ownership.

Issues

1. Because the check for locked is done using the current copy of the article, multiple users may see the article as unlocked.
1. Because of (1) multiple users will lock the article, last man wins but the others think the won.
1. Either by bad code, or because they erroneously think they have the lock, anyone can release the lock when the don’t own it.

Recommended changes:

1. Store the DateTime the lock expires DateTime so the stored procedures can check for expired lock.

1. Instead of using art_ArticleUpdateLockInfo stored procedure
UPDATE Article
SET EditLockGained = @EditLockDate, EditLockMemberId = @EditingMemberId
WHERE ArticleID = @ArticleID

or create new art_ArticleGetLock stored procedure
UPDATE Article
SET EditLockExpires = @EditLockDate, EditLockMemberId = @EditingMemberId
WHERE ArticleID = @ArticleID and
((editLockMemberId is null or editLockMemberId = @EditingMemberId) or
(editLockExpires is null or editLockExpires < GetDate()))

Success can be check by the number of records updated == 1

1. Similarily, an art_ArticleReleaseEditLock should be
UPDATE Article
SET EditLockExpires = null, EditLockMemberId = null
WHERE ArticleID = @ArticleID and
( editLockMemberId = @EditingMemberId or editLockExpires < GetDate()))

Success can be check by the number of records updated == 1

There was a bug in EnsureFilesInSourceControl (or something like it) that was preventing updating of the Git if the Workspace already existed. Fixed this earlier.

The order by Popularity and limit of 20 article was killing the Showcase article as they have a low popularity (probably 0 or null).

Also the Poll was not being shown in the Monday Daily Build.

NewsletterRss.Aspx filters articles votes>=3 and rating >=3.0.
Showcase articles don't show the rating control, but surprisingly do show the Reporting flag. This means that the article will never be sent as content.
Additionally, only the top 20 articles, ordered by Popularity are selected.

Chris, how do you want to change this?

I may have inadvertently left the Incremental Crawl stopped, but will monitor for a bit to see if this re-occurs.

Actually it does, but was not pushing to all servers.

The Question is the Thread Message, so we need to allow tags when the initial question is posted.

Now we need to distinguish between the initial message and the replies.

We could use Search to get a list of the Questions and use the Thread View page to display the replies.

probably the best thing to do is set the Save Filters On and hide it.

Still setting the allTags flag!!!

Arrghhh! page needed compiling. ok now.

When using Search for the Question List, allTags is set to false.

document images are being converted correctly.
Changed code to convert Simley src URLs to absolute, but Workspaces is converting the http://www.codeproject.com to http://workspaces.codeproject.com for these urls.

Now a Super Task for the tasks to complete all this.

A Union is much faster than an or in the Where clause.

I think that Runtime has to try and log the user in using CodeProject cookies.

On file save error, such as invalid exception, or other reasons, the filename to use for creating the RelatedFile.FileUrl was set to the original HttpFile filename, not the local filename. Then, the code did not check the Result.Success value when looping through the result collection from saving the files.

This has to be done by the someone who has rights to the CodeProject LinkedIn account.

Message were being removed from the faulted queue but not re-inserted if updating the cache failed.

Changed text in Draft mode to "Exit without Publishing"

changed to use System.IO.Compression.ZipArchive.
This seems to create valid zips.
Removed RAR from the lists of valid file types.

API/Authentication/Register was incorrectly returning and error of 'invalid operation' where the email was new, but the username was a duplicate. This was traced to an invalid assumption in the UserRepository.UserNameAvailable method which assumed that if UserProfile did not exist in the cache, it didn't exist. Changed to check the database as well.

not required. Can determine files from RelatedFiles.

/Api/Git/GetAuthorizationInfo already exists.

IncreaseDownloadCount existed with additional parameter which has been removed and tested.

"I acknowledge that information from my Professional Profile may be sent to third party sponsors of Research Library content."

The checkbox should be marked as required.

Forget the stuff in parentheses. Makes it too long.

The methods using the UserCacher.CacheUserInfo did not ensure that the UserProfile had all the properties populated. TagList, for one, was null in may cases. This has been corrected.

Test message -- Ignore

The UserCacher does not cache tags.
Should this be rewritten to use the new MappingCacher?

Haven't decided where is the best place for these files yet.

Not so. The current implementation can use any unique string as an ItemReference. However, if it is in the format of site.objectType.Id, the additional information is parsed and saved. I think the issue is that other services are assuming that the siteId, objectTypeId, and objectId exist and are using them to join tables rather that having a GlobalItemId.

Test Comment

After discussion with Kamil, made sure that CodeProject only displays Comments for Items on the Site, not all.
Does not require Private flag on Comment. Private/Public is a Workspace attribute, not a Comment attribute.

I used an array of ints of 5 digit length as the product would be 10 digits or less, the max on the machine we were using. Then iteratively multiplied the array by the each number and 'normalized' the array items to 5 digits. See my alternate below on using the BigInteger class.

I had an assignment in university to calculate 72!, all 100 digits of it. Most of the class blew their account limits with infinite loops and other errors.

x

asfasf asdfasf

Reason for my vote of 5
test vote

Reason for my vote of 3
test vote

this is a test comment

test

test

You might want to check out the Reactive Extensions for .Net (or JavaScript). Any IEnumerable can be turned into an Observable collection. See http://msdn.microsoft.com/en-us/devlabs/ee794896.aspx.
There are also a number of Blogs about it. Also Channel9 has several videos.

this is a test

x

test to see iff the spaces get mangled.

test "test";)

test

A test comment; alskfjslfkj

This is a test for a commenting issue, ignore.

asfdasfd

Testing Comment