I'm getting two errors while syncing:

  1. Unknown column "fa.id", which is coming from an incorrect table alias getting set
  2. Problems with the loadPropertySet() function
CommentFileSizeAuthor
#7 dynamic-forms-3013742-7.patch8.95 KBfathershawn
#5 dynamic-forms-3013742-5.patch8.23 KBfathershawn
#4 errors-syncing-3013742-4.patch9.11 KBfathershawn
#2 delete-forms-3013742-2.patch1.74 KBkarlshea

Comments

KarlShea created an issue. See original summary.

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Status: Active » Needs review
StatusFileSize
new delete-forms-3013742-2.patch1.74 KB

Patch attached

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Title: Multiple errors syncingforms » Multiple errors syncing forms
fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
StatusFileSize
new errors-syncing-3013742-4.patch9.11 KB

I've moved some of the work into the database and refactored to enable/disable given changed in the FA since this module was first built in D7. You'll need a drush entup after applying the patch.

fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
StatusFileSize
new dynamic-forms-3013742-5.patch8.23 KB

Reroll on coding standards update

Status: Needs review » Needs work

The last submitted patch, 5: dynamic-forms-3013742-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
Status: Needs work » Needs review
StatusFileSize
new dynamic-forms-3013742-7.patch8.95 KB

Reroll - coding standards

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

Appears to work for me!

The last submitted patch, 2: delete-forms-3013742-2.patch, failed testing. View results

The last submitted patch, 4: errors-syncing-3013742-4.patch, failed testing. View results

  • FatherShawn committed 04d788b on 8.x-1.x 
    Issue #3013742 by KarlShea, FatherShawn: Multiple errors syncing forms
fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.