CodeIgniter 1.7.0 Released

[eluser]DiRN[/eluser]
[quote author="dmiden" date="1225076954"]While this is true that nowhere in the documentation does it say that ASC, DESC, or RANDOM is required, it applies to the valid syntax as outlined in the documentation.

Code:

// And using the syntax:
$this->db->order_by('field1, field2'); // is not a valid syntax.
// With 1.7 the valid syntax would be:
$this->db->order_by('field1', 'desc');
$this->db->order_by('field2', 'desc');
// Second parameter options are asc or desc, or random.
// OR:
$this->db->order_by('field1 desc, field2 desc');
// Using ('field option, field2 option2').

[/quote]

The documentation clearly states

You can also pass your own string in the first parameter

That is exactly what I'm doing.

While the example provided by the documentation uses DESC and ASC, there is no mention that they are required (as they shouldn't be seeing as how ASC and DESC aren't required by any SQL implementation I've ever seen). I took the example and omitted a modifier because the default behavior (according to SQL syntax) suited my needs.

I think my issue here is that I've stumbled upon two significant changes to the core of CI that break my existing code (I would imagine many other developers will have these problems as well), neither of which is mentioned in the upgrade documentation, nor does my code violate anything stated in the documentation other than examples. Maybe that's what the examples are meant to do, but I've never seen documentation before that relied on its examples rather than its text to define all valid uses.