Add ability to drag drive to terminal

[X512]

After dropping, path to device (/dev/disk/...) should be pasted to terminal. It will be useful for commands like «dd». Also Cmd+C should copy path to selected device to clipboard.

[jwlh172]

Patch with drag and drop functionality.

[jwlh172]

I've created a patch adding requested drag and drop functionality.

[stippi]

Patch looks fine in terms of functionality. You can improve the code a bit in terms of simplifying it and making it more coding style compliant:

BRow* _selectedRow = RowAt(rowPoint); 
if(_selectedRow) { 
    PartitionListRow* selectedRow 
        = dynamic_cast<PartitionListRow*>(_selectedRow); 

You can simplify that as:

   PartitionListRow* selectedRow 
       = dynamic_cast<PartitionListRow*>(RowAt(rowPoint));
   if (selectedRow != NULL) {

It is safe to pass NULL to dynamic_cast. In addition, you also catch the case where the returned BRow is in fact not of type PartitionListRow in which case dynamic_cast returns NULL which the original did not check for. BTW, I would call the variable "draggedRow" instead of "selectedRow". Makes it clearer, IMHO.

The same change is applicable for the BStringColumn retrival. Also note the space after "if".

What I don't like so much about the implementation is how it mixes model specific knowledge with view specific knowledge. But it's been very long since I looked at or even wrote the code, so it may perfectly fit with the rest of PartitionListView.cpp, I don't remember. At the same time I am not a fan of making things overlay complicated. But if there is an easy alternative to your solution that better separates the model from the view logic, then I'd prefer that. (The list view would forward the event to the model class, if there even is such a thing, and for example let it fill out the contents of the drag BMessage.)

Thanks for working on this!

[leavengood]

This file does not seem particularly MVC to me, so I think the implementation in the patch is fine.

[jwlh172]

Additional patch; to be applied following 0001.

[jwlh172]

• Forgot to add spaces after if statements.

[jwlh172]

I've simplified the code where possible and, in an attempt to adhere at least a bit more to MVC, I've added public method GetPath() to the PartitionListRow class, so now I simply call that method from my PartitionListView in order to receive the associated drive/partition path.

[stippi]

Thanks for the update. Sorry, I've got more improvements to suggest:

const char* 
PartitionListRow::GetPath()
{ 
    char* pathString = NULL;

    BBitmapStringField* stringField
        = dynamic_cast<BBitmapStringField*>(GetField(kDeviceColumn));
    if(stringField != NULL) {
        pathString = const_cast<char*>(stringField->String());
    }

    return pathString;
}

According to Haiku's conventions, the method should be named just "Path()". Maybe "DevicePath()" would be clearer. Since it doesn't change the object, it should be declared const. When you return "const char*", I don't understand why you declare your return variable "char*" and then even need the const_cast to make the compiler happy. You can avoid the return variable entirely using an early return:

const char* 
PartitionListRow::DevicePath() const
{ 
    BBitmapStringField* stringField
        = dynamic_cast<BBitmapStringField*>(GetField(kDeviceColumn));
    
    if (stringField == NULL)
        return NULL; // or return ""; ?

    return stringField->String();
}

Can you provide that as a single patch? Again, thanks a lot for your work!

[jwlh172]

New patch; replaces all above patches; all together in this one.

[jwlh172]

I've implemented the latest suggestions from stippi. The newest attached patch includes all modifications.

[stippi]

Committed in hrev44647. Sorry for the huge delay and thanks again!