View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
0000928 | v4.0 Release (Planning) | Core Infrastructure | public | 2011-08-21 07:51 | 2016-12-29 10:16 | ||||||||
Reporter | caseydk | ||||||||||||
Assigned To | caseydk | ||||||||||||
Priority | normal | Severity | feature | Reproducibility | always | ||||||||
Status | acknowledged | Resolution | fixed | ||||||||||
Product Version | |||||||||||||
Target Version | 4.0 | Fixed in Version | 4.0 | ||||||||||
Summary | 0000928: Fluent Interfaces for w2p_Database_Query | ||||||||||||
Description | Right now most of w2p_Database_Query's methods don't return anything. Since we usually use a whole series of them in sequence, a fluent interface would make the entire process simpler *and* clearly group things together as a single process. | ||||||||||||
Additional Information | Example: // Fluent interface $myCar = new Car(); $myCar->setSpeed(100)->setColor('blue')->setDoors(5); // Example without fluent interface $myCar2 = new Car(); $myCar2->setSpeed(100); $myCar2->setColor('blue'); $myCar2->setDoors(5); Source: http://en.wikipedia.org/wiki/Fluent_interface | ||||||||||||
Tags | No tags attached. | ||||||||||||
Attached Files |
|
![]() |
|
w2p_user (reporter) 2011-08-24 01:11 |
I dont think it would be helpful during debugging of issues in many ways going by what i see on wikipedia. "This style is marginally beneficial in readability due to its ability to provide a more fluid feel to the code[citation needed]. However, it can be detrimental to debugging, as a fluent chain constitutes a single statement for which debuggers may not allow setting up intermediate breakpoints, for instance." |
caseydk (administrator) 2014-10-04 22:11 |
Resolved in v4.0 development |
caseydk (administrator) 2016-12-29 10:16 |
I'm changing this issue to acknowledged. While it's resolved in the v4.0 development branch. That is going to be reviewed commit by commit and merged into a new development branch over the next couple months. |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2011-08-21 07:51 | caseydk | New Issue | |
2011-08-21 07:52 | caseydk | Status | new => assigned |
2011-08-21 07:52 | caseydk | Assigned To | => bigbluehat |
2011-08-24 01:11 | w2p_user | Note Added: 0002160 | |
2011-09-27 23:49 | caseydk | Project | v3.0 Release => Pending Requests |
2014-01-01 17:02 | caseydk | Project | Pending Requests => v4.0 Release (Planning) |
2014-01-21 17:57 | caseydk | Target Version | => 4.0 |
2014-03-30 22:12 | caseydk | Category | General => Core Infrastructure |
2014-08-16 22:36 | caseydk | Assigned To | bigbluehat => caseydk |
2014-10-04 22:11 | caseydk | Note Added: 0003655 | |
2014-10-04 22:11 | caseydk | Status | assigned => resolved |
2014-10-04 22:11 | caseydk | Fixed in Version | => 4.0 |
2014-10-04 22:11 | caseydk | Resolution | open => fixed |
2016-12-29 10:16 | caseydk | Status | resolved => acknowledged |
2016-12-29 10:16 | caseydk | Note Added: 0003834 |