It would be really handy to add Drush support.

To be able create, update, reset views.

CommentFileSizeAuthor
#13 interdiff-2462241-12-13.txt3.09 KBmrded
#13 views_import-2462241-13.patch3.7 KBmrded
#12 interdiff-2462241-7-12.txt618 bytesmrded
#12 views_import-2462241-12.patch3.66 KBmrded
#7 drush_support-2462241-6.patch3.29 KBbigjim
#6 drush_support-2462241-5.patch1.68 KBbigjim
#4 drush_support-2462241-4.patch2.79 KBbigjim
#3 drush_support-2462241-3.patch1.63 KBbigjim
#2 drush_support-2462241-2.patch1.63 KBbigjim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bigjim’s picture

bigjim CreditAttribution: bigjim commented

The last one you can accomplish already, you can reset Views using the vr command that ships with views

ie:
drush vr <view_name>
or
drush vr --all

bigjim’s picture

bigjim CreditAttribution: bigjim commented
FileSize
drush_support-2462241-2.patch1.63 KB

Wrote a patch to support exporting files using drush.

Adds the vie command which supports options for writing individual views or all the views in the system.

Usage
To export all views to views_import folder:

drush vie --all

To export the frontpage view to views_import folder:

drush vie frontpage
bigjim’s picture

bigjim CreditAttribution: bigjim commented
FileSize
drush_support-2462241-3.patch1.63 KB

Opps wrong patch :D

bigjim’s picture

bigjim CreditAttribution: bigjim commented
FileSize
drush_support-2462241-4.patch2.79 KB

More updates, little things like including the drush callback function. ;)

mrded’s picture

mrded CreditAttribution: mrded commented
Status: Active » Needs work

@bigjim, thanks a lot for your patch, that's really helpful!

Can you also fix a code style if you don't mind? Or I can do it myself.

bigjim’s picture

bigjim CreditAttribution: bigjim commented
FileSize
drush_support-2462241-5.patch1.68 KB

This should do it.

I added a few fixes to issues already there ;)

bigjim’s picture

bigjim CreditAttribution: bigjim commented
FileSize
drush_support-2462241-6.patch3.29 KB

oops forgot the new files again (Git and I aren't friends this week)

bigjim’s picture

bigjim CreditAttribution: bigjim commented
Status: Needs work » Needs review

updating to needs review

AlexKirienko’s picture

AlexKirienko CreditAttribution: AlexKirienko at WikiJob commented

@bigjim Great patch! Thank you

mrded’s picture

mrded CreditAttribution: mrded commented
Status: Needs review » Needs work

Looks like VIEWS_IMPORT_DIR constant is not visible from views_import.drush.inc file.

drush vie --all
file_put_contents(VIEWS_IMPORT_DIR/reviews.inc): failed to open      [warning]
stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/nodes.inc): failed to open stream:[warning]
No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/archive.inc): failed to open      [warning]
stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/comments_recent.inc): failed to   [warning]
open stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/frontpage.inc): failed to open    [warning]
stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/glossary.inc): failed to open     [warning]
stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/taxonomy_term.inc): failed to open[warning]
stream: No such file or directory views_import.drush.inc:50
file_put_contents(VIEWS_IMPORT_DIR/tracker.inc): failed to open      [warning]
stream: No such file or directory views_import.drush.inc:50
mrded’s picture

mrded CreditAttribution: mrded commented

One more question: Why do you use --all=1 argument? Why not just --all?

mrded’s picture

mrded CreditAttribution: mrded commented
FileSize
views_import-2462241-12.patch3.66 KB
interdiff-2462241-7-12.txt618 bytes

I made a fix of VIEWS_IMPORT_DIR visibility.

mrded’s picture

mrded CreditAttribution: mrded commented
Status: Needs work » Needs review
FileSize
views_import-2462241-13.patch3.7 KB
interdiff-2462241-12-13.txt3.09 KB

I also made a quick refactoring, If you are interested.

  • mrded committed e0a8699 on 7.x-1.x authored by bigjim 
    Issue #2462241 by bigjim, mrded: Drush support

  • mrded committed 1b3da58 on 7.x-1.x 
    Issue #2462241 by mrded: Refactor drush support
mrded’s picture

mrded CreditAttribution: mrded as a volunteer and at WikiJob for WikiJob commented
Status: Needs review » Fixed

@bigjim, Thank you for your work! Great patch ;)

Status: Fixed » Closed (fixed)

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