I've faced with Fatal error during migration Redirects due to incorrect field type for redirect_options and source_options. In schema those fields have type 'text' I think will be better to set it to "BLOB".

CommentFileSizeAuthor
#8 redirect-redirect_options-source_options-should-be-blob-2828083-8.patch1.2 KBandriy khomych
#3 redirect-redirect_options-source_options-should-be-blob-2828083-3.patch1.2 KBandriy khomych
#2 redirect-redirect_options-source_options-should-be-blob-2828083-2.patch551 bytesandriy khomych

Issue fork redirect-2828083

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ozin created an issue. See original summary.

andriy khomych’s picture

andriy khomych commented
StatusFileSize
new redirect-redirect_options-source_options-should-be-blob-2828083-2.patch551 bytes

Added patch, please check it.

andriy khomych’s picture

andriy khomych commented
StatusFileSize
new redirect-redirect_options-source_options-should-be-blob-2828083-3.patch1.2 KB

Updated patch.

andriy khomych’s picture

andriy khomych commented
Status: Active » Needs review
br0ken’s picture

br0ken
Primary language Ukrainian
commented 
+++ b/redirect.install
@@ -432,3 +432,11 @@ function _redirect_migrate_path_redirect_variables() {
+ * Change fields 'redirect_options', 'source_options' to blob type

CS issue: missing dot at the end of sentence.

andriy khomych’s picture

andriy khomych commented
Assigned: Unassigned » andriy khomych
andriy khomych’s picture

andriy khomych commented
Status: Needs review » Active
andriy khomych’s picture

andriy khomych commented
Assigned: andriy khomych » Unassigned
Status: Active » Needs review
StatusFileSize
new redirect-redirect_options-source_options-should-be-blob-2828083-8.patch1.2 KB

Added updated patch.

br0ken’s picture

br0ken
Primary language Ukrainian
commented
Status: Needs review » Reviewed & tested by the community

Good improvement.

ozin’s picture

ozin
Primary language Ukrainian
commented

Looks good, great job!

pifagor’s picture

pifagor commented
pifagor’s picture

pifagor commented

RTBC

alex_optim’s picture

alex_optim
Location Tallinn
commented

+1

pifagor’s picture

pifagor commented

  • pifagor committed b8561a8 on 7.x-1.x 
    Issue #2828083 by Andriy Khomych, pifagor, ozin, BR0kEN, alex_optim:...
pifagor’s picture

pifagor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

stedmanj made their first commit to this issue’s fork.