DBTNG: Block module

I think is better to concentrate efforts first in #257032: Split block $ops into separate functions

Reactiviating.

Wow. Duplicated efforts:

#302263: Rework block.module for the new data api

@hswong3i: If you want to compare notes or work together on this somehow, let me know. Both our patches need to be re-rolled.

But maybe dropcube is right about #257032: Split block $ops into separate functions

Ok, another patch, also converted the dynamic queries and added a tag to the block loading query.

Committed to CVS HEAD. Thanks!

Why are we having explicit int casts for default values? They were not needed before. That's curious.

FALSE and NULL no longer auto-fold to int 1, so that has to be done explicitly. I do not want to do that automatically because of the performance hit.

But why in #default_value and not in the save routines...? And, in general, if we are using d_w_r which retrieves the schema anyways, then we could autocast there.

Not sure, the first one is obviously needed, but I can't find a reason for these, manual testing and the block tests work, let's ask the test bot...

Ok, so that obviously doesn't work, not sure why my local tests did work though. Another try which converts the value only before saving...

This time for real, tests pass locally...

There should be a space between the (int) and the $var.

Re-roll with casting coding style fixed.

Looks good.

Committed to CVS HEAD. Thanks.