Comments (2)
It is better just to call delete_records, then return true.
If delete_records has an error, it will throw an exception, otherwise it always returns true.
If the record doesn't exist, we have nothing to delete, so we can safely return true, as the clean-up has already been done.
I've a patch locally to fix this, so I'll open a pull request when testing is done.
from moodle-mod_collaborativefolders.
Fixed via #34
from moodle-mod_collaborativefolders.
Related Issues (20)
- Task does not check whether coursemodule still exists HOT 2
- Even if no technical user is registered, teachers can create mod instances without any special hint
- task realises rather late that no technical user is logged in HOT 2
- Settings: "Manage the technical user" does not indicate the current user HOT 1
- Migrate to M3.3 OAuth API HOT 3
- [Fake issue for README.md images] HOT 1
- Issues in Moodle that cause problems for this plugin
- Enforce a folder structure that supports more than one application HOT 5
- Store the ownCloud username after logging in HOT 1
- After redirect, highlight the active folder
- Implement the privacy API HOT 1
- Decouple from specific cloud service name. HOT 1
- The "isteacher" capability name might be confusing HOT 1
- More explicit selectors in styles.css HOT 3
- Verbindung zur Nextcloud klappt nicht HOT 3
- The system account is unable to connect to NextCloud, so folders for this activity will not be created. Please inform the administrator about this.
- error/Could not upgrade oauth token HOT 9
- Wrong language entry in "openinowncloud" HOT 2
- Add an admin setting to control the default for teacher access
- Bug: share not created with long course name
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from moodle-mod_collaborativefolders.