We should be able to export section URL aliases through features. Right now there is no alias table in the db.

CommentFileSizeAuthor
#8 section-settings_blob-1472774-8.patch3.37 KBmarkusbroman
#6 section-settings_blob-1472774-6.patch4.05 KBmarkusbroman
#1 section-export_aliases_patch-1472774-1.patch2.38 KBmarkusbroman

Comments

markusbroman’s picture

markusbroman commented
Status: Fixed » Needs review
StatusFileSize
new section-export_aliases_patch-1472774-1.patch2.38 KB
logaritmisk’s picture

logaritmisk commented

It would be a better idea to create a data blob column, instead of a alias column, that can be used to save things like path aliases and maybe menu items to support a nice clean export.

anon’s picture

anon
Primary language Swedish
Location Gothenburg
commented
Status: Needs review » Needs work

Yes, a blob would be the perfect way to deal with this.

fabsor’s picture

fabsor commented
Title: Featurize aliases » Add a blob for arbitrary settings like path aliases and menu items

Changing title to something more appropriate =)

fabsor’s picture

fabsor commented
Title: Add a blob for arbitrary settings like path aliases and menu items » Add a blob column for arbitrary settings like path aliases and menu items

Even better.

markusbroman’s picture

markusbroman commented
Status: Needs work » Needs review
StatusFileSize
new section-settings_blob-1472774-6.patch4.05 KB

Updated patch with blob column

fabsor’s picture

fabsor commented
Status: Needs review » Needs work

You got some additions to the info file that probably got in by accident. Those lines are added automaticly by the drupal.org packaging system.

+++ b/section.infoundefined
@@ -6,3 +6,10 @@ core = 7.x
 dependencies[] = ctools
+
+; Information added by drupal.org packaging script on 2012-03-10
+version = "7.x-1.x-dev"
+core = "7.x"
+project = "section"
+datestamp = "1331339933"

There are also a couple of whitespace issues in this patch that could be fixed before it goes in. I guess the implementation is OK for now, but we will probably have to create our own entity controller in the long run. But I would say that's a separate issue.

markusbroman’s picture

markusbroman commented
Status: Needs work » Needs review
StatusFileSize
new section-settings_blob-1472774-8.patch3.37 KB

Thanks fabsor, I have fixed the whitespace issues and the automated lines of code. This patch applies to the latest 7.x-1.x version

randallknutson’s picture

randallknutson commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

Closing out old issues.