Column not found: 1054 Unknown column 'page'

[alanmels]

Hello Jen,

After accepting the last three PRs, I noticed that categories can no longer be created or edited — the system throws an error. Rather than rolling back those commits, I was hoping you might be able to take a look and fix this based on the current state of the dev branch. Unfortunately, I won’t have time to address it myself soon.

Thanks,
Alan

PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'page' in 'SET': UPDATE {simplenews_category} SET `tid`=:db_update_placeholder_0, `format`=:db_update_placeholder_1, `priority`=:db_update_placeholder_2, `receipt`=:db_update_placeholder_3, `from_name`=:db_update_placeholder_4, `email_subject`=:db_update_placeholder_5, `from_address`=:db_update_placeholder_6, `hyperlinks`=:db_update_placeholder_7, `new_account`=:db_update_placeholder_8, `opt_inout`=:db_update_placeholder_9, `block`=:db_update_placeholder_10, `page`=:db_update_placeholder_11 WHERE ( (`tid` = :db_condition_placeholder_0) ); Array ( [:db_update_placeholder_0] => 1 [:db_update_placeholder_1] => plain [:db_update_placeholder_2] => 0 [:db_update_placeholder_3] => 0 [:db_update_placeholder_4] => My Backdrop Site [:db_update_placeholder_5] => [[simplenews-category:name]] [node:title] [:db_update_placeholder_6] => noreply@ddev.site [:db_update_placeholder_7] => 1 [:db_update_placeholder_8] => none [:db_update_placeholder_9] => double [:db_update_placeholder_10] => 1 [:db_update_placeholder_11] => 1 [:db_condition_placeholder_0] => 1 ) in simplenews_category_save() (line 1900 of /var/www/html/docroot/modules/contrib/simplenews/simplenews.module). 

[jenlampton]

Sure thing. I ran into this too. Let me see what I did to fix it (I think it might have been running the update.php again, but I may have also changed something in the update hook... )

[jenlampton]

Ah, When I applied the patch, the update number was 1002 at first, but since a previous patch I applied also included an update numbered 1002, I had to change the number to 1003 and run the update again.

In the 1.x-1.x branch, the last update number is 1001. 1002 is the next one in the sequence, so it should work if you had not previously applied any other patches that included an update with that number.

Can you do a fresh install of the 1.x-1.2.1 release, and the update to 1.x-1.x, and see if you still have the problem? That worked as expected for me.

[alanmels]

The error is persisting on a fresh setup, also giving: Warning: Undefined property: stdClass::$page in simplenews_category_save() (line 1898 of /var/www/html/docroot/modules/simplenews/simplenews.module).

[alanmels]

Hello @jenlampton, I had to revert the last commit to make the module's development branch functional. Unfortunately, I do not have time to troubleshoot this, so I hope you can take another stab at implementing this new feature without causing errors.

[jenlampton]

Hi @alanmels it looks like the PR I'd added didn't actually include the update hook, which explains all the problems you were having. I've pushed an updated version to #70.

[alanmels]

Sorry, but it's giving the following error on module enable:

Error
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'page' cannot be null: INSERT INTO {simplenews_category} (`tid`, `format`, `priority`, `receipt`, `from_name`, `email_subject`, `from_address`, `hyperlinks`, `new_account`, `opt_inout`, `block`, `page`) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7, :db_insert_placeholder_8, :db_insert_placeholder_9, :db_insert_placeholder_10, :db_insert_placeholder_11); Array ( [:db_insert_placeholder_0] => 1 [:db_insert_placeholder_1] => plain [:db_insert_placeholder_2] => 0 [:db_insert_placeholder_3] => 0 [:db_insert_placeholder_4] => My Backdrop Site [:db_insert_placeholder_5] => [[simplenews-category:name]] [node:title] [:db_insert_placeholder_6] => noreply@ddev.site [:db_insert_placeholder_7] => 1 [:db_insert_placeholder_8] => none [:db_insert_placeholder_9] => double [:db_insert_placeholder_10] => 1 [:db_insert_placeholder_11] => )