Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

button 1 = delete plugin (all functions) #18

Open
Tracked by #12
aljawaid opened this issue Jul 7, 2023 · 6 comments
Open
Tracked by #12

button 1 = delete plugin (all functions) #18

aljawaid opened this issue Jul 7, 2023 · 6 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@aljawaid
Copy link
Owner

aljawaid commented Jul 7, 2023

No description provided.

aljawaid added a commit that referenced this issue Jul 9, 2023
@aljawaid aljawaid added the enhancement New feature or request label Jul 10, 2023
aljawaid added a commit that referenced this issue Jul 12, 2023
@aljawaid aljawaid self-assigned this Jul 14, 2023
@aljawaid aljawaid added this to the Initial Release milestone Jul 14, 2023
@aljawaid aljawaid added the help wanted Extra attention is needed label Jul 14, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 23, 2023
aljawaid added a commit that referenced this issue Jul 30, 2023
@aljawaid
Copy link
Owner Author

@alfredbuehler @creecros Hello, this is the last issue for this plugin and it is ready for the release finally, but I have a question...

All the individual steps for deleting a plugin have been done. Now they all need to be completed through the 'delete plugin' button, which does all the parts (schema, tables, entries etc) and totally removes the plugin.

Is this model function correct with the controller? I'm struggling to understand how to combine all the other functions into the delete plugin function:

Modal Button:

<?= $this->modal->confirmButtons('PluginCleaningController', 'deletePlugin', array('plugin_job_name' => $plugin_job_name, 'plugin' => 'ContentCleaner'), t('Delete Plugin')) ?>

Controller:

public function deletePlugin($plugin_name)

Model:

public function deletePlugin($plugin_name)

Could you kindly check and advise? I think my model function is wrong but I am not exactly sure what to do next and why it is wrong. I just need one or both of your expert opinions.

@alfredbuehler
Copy link
Collaborator

I'm currently on vacation, but I'll might have a look into it next week.

@aljawaid
Copy link
Owner Author

I'm currently on vacation, but I'll might have a look into it next week.

thanks, I look forward to it. Have a nice holiday.

aljawaid added a commit that referenced this issue Jul 31, 2023
aljawaid added a commit that referenced this issue Jul 31, 2023
@alfredbuehler
Copy link
Collaborator

All the individual steps for deleting a plugin have been done.
Are they tested and working?

Is this model function correct with the controller? I'm struggling to understand how to combine all the other functions into the delete plugin function:

There is indeed some weirdness in using the controller function and downwards.

@alfredbuehler
Copy link
Collaborator

There are issues in front of the controller. If I try to delete the installed AddressBook plugin, the donut rotates forever, but the controller function deletePlugin is never called. I cannot debug this. And the amount of work to fix or complete the overall deleting is far too huge for me.

@aljawaid
Copy link
Owner Author

aljawaid commented Aug 4, 2023

ok no problem, thanks for trying

aljawaid added a commit that referenced this issue Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants