U4-8259 - NPoco - run on stock NPoco

Created by Stephan 30 Mar 2016, 14:57:05 Updated by Shannon Deminick 01 Jun 2016, 14:24:17

We should try to run on stock NPoco instead of using our own current version.

At the moment there is only one issue: the OrderBy fields need to be properly aliased (eg "ContentDto__NodeDto__Id" vs "umbracoNode.Id") for paging queries to work properly with SqlServer. See discussion at https://github.com/schotime/NPoco/issues/248.

Comments

Stephan 11 Apr 2016, 06:39:09

Update:

  • the OrderBy issue can be managed at Umbraco level without changes in NPoco - solved.
  • need a "Sql with Context" for U4-8287 - currently discussing w/Adam to have it in NPoco.


Shannon Deminick 11 May 2016, 08:55:28

@zpqrtbnk is this one complete now?


Stephan 17 May 2016, 07:09:04

Currently running on 3.3.0-beta3 and will run on 3.3.0 when it's out. Keeping this issue open until then.


Priority: Normal

Type: Task

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions:

Due in version: 8.0.0

Sprint: Sprint 16

Story Points:

Cycle: