Skip to content
This repository has been archived by the owner on Jun 27, 2024. It is now read-only.

Commit

Permalink
Fixed Video duration
Browse files Browse the repository at this point in the history
- Corrección en el tiempo cuando se agrega una canción a la cola. #7

- Corrección en las imagenes y ejemplos del readme.
  • Loading branch information
DavichoStar committed Mar 4, 2021
1 parent fed306f commit 5138339
Show file tree
Hide file tree
Showing 10 changed files with 68 additions and 42 deletions.
22 changes: 16 additions & 6 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -1,30 +1,40 @@
---
name: Reporte de Bug 🐞
about: Crea un reporte claro para ayudarnos a resolverlo.
title: '[BUG ERROR WARN] - Titulo'
title: '[BUG WARNING] - Title'
labels: 'Type: Bug'
assignees: ''
---

### Descripción del problema
### Description of the problem

<!-- Inserta una descripcion clara de que está ocurriendo -->

...

### Pasos para reproducir
### Steps to reproduce

<!--- Aqui va el paso a paso de como logras generar el error
Asegurate de cubirlo todo sin ambiguedades -->

...

### Resultado esperado
### Expected result

<!-- Que se supone que deberías estar recibiendo o pasando. -->

...

### Resultado real
### Actual result

<!-- Aqui coloca que es lo que terminas recibiendo o el error final que se te arroja. -->

...

---
### Logs e informacion relevante

### Logs and relevant information

<!-- Incluye aqui un detalle de archivos y referencias que nos puedan ayudar a resolver el problema -->

...
6 changes: 3 additions & 3 deletions .github/ISSUE_TEMPLATE/config.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
blank_issues_enabled: false
contact_links:
- name: Discussions
url: https://github.com/faztcommunity/xfazt-bot/discussions
about: Considera colocar nuevas ideas en la sección de discusiones
- name: Discussions
url: https://github.com/StarArmyDev/starmusic/discussions
about: Considera colocar nuevas ideas en la sección de discusiones
17 changes: 12 additions & 5 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -1,19 +1,26 @@
---
name: Nueva funcionalidad o ideas 💡
about: Para incorporar nuevas funciones o sugerir ideas que puedan implementarse crea un ISSUE de este estilo.
title: '[IDEA SUGERENCIA IMPLEMENTACION] - Titulo'
title: '[FEATURE REQUEST] - Title'
labels: 'Type: Enhancement'
---

<!-- Considera usar Discussions: https://github.com/faztcommunity/xfazt-bot/discussions?discussions_q=category%3AIdeas -->

### Descripción de la funcionalidad
### Description of functionality

<!-- Inserta una descripcion de lo que tienes pensado lograr. -->

...

### Propuesta de implementación
### Implementation proposal

<!-- Aqui dinos como esperas lograrlo (por ejemplo que archivos, librerias, funciones utilizarás, patrones de diseño, etc) -->

...

### Screens / Maquetado idea
<!-- Si ya tienes la implementación hecha o algún grafico que muestre lo que tienes, por favor colócalo por aqui. -->
### Screens / Idea layout

<!-- Si ya tienes la implementación hecha o algún gráfico que muestre lo que tienes, por favor colócalo por aqui. -->

...
18 changes: 10 additions & 8 deletions .github/ISSUE_TEMPLATE/requirement.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,16 +5,17 @@ title: 'Se requiere <titulo de la funcionalidad>'
labels: 'Type: Enhancement'
---

<!-- Aunque no son lo mismo requisito suele escribirse como una historia de usuario,
<!-- Aunque no son lo mismo requisito suele escribirse como una historia de usuario,
Para más información lee: https://scrummanager.net/files/historias_usuario_scrum_manager.pdf
-->

### \<Titulo de la funcionalidad>
* Como: *rol*,
* quiero: *objetivo*,
* para poder: *beneficio*.
### \<Feature title>

- Como: _rol_,
- quiero: _objetivo_,
- para poder: _beneficio_.
<!--La prioridad puede ser reemplazada por un label-->
* Prioridad: *baja, media, alta, muy alta*
- Prioridad: _baja, media, alta, muy alta_

<!--Ejemplo:
Como astronauta
Expand All @@ -23,12 +24,13 @@ para desarrollar una nueva colonia.
prioridad: muy alta
-->

### Details

### Detalles
<!-- Para efectos prácticos del issue, se puede detallar hasta casos de uso y reglas de negocio. El equipo se encargará de subdividir en nuevos casos de usos.
Esta sección es recomendable, pero opcional.
-->
*\<Considerando qué/En relación a> \<caso>*, es *\<necesario/recomendable/podría implementarse> \<objetivo>* para *\<beneficio> \<prioridad>*. *\<explicar el cómo con detalles>*

_\<Considerando qué/En relación a> \<caso>_, es _\<necesario/recomendable/podría implementarse> \<objetivo>_ para _\<beneficio> \<prioridad>_. _\<explicar el cómo con detalles>_

<!--Ejemplo:
Considerando que el planeta se está quedando sin recursos, es necesario construir un cohete para ir a marte urgentemente. El cohete debe tener 50 metros de alto, y 9 de diametro, construido en algún material resistente. El astronauta debe subirse y llegar a salvo a su destino, además requiere almacenamiento para herramientas de minería.
Expand Down
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,13 @@ y este proyecto se adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.

## [Unreleased]

## [v2.0.1] - 2021-03-04

### Fixed

- Corrección en el tiempo cuando se agrega una canción a la cola.
- Corrección en las imagenes y ejemplos del readme.

## [v2.0.0] - 2021-02-22

## Added
Expand Down Expand Up @@ -77,3 +84,4 @@ y este proyecto se adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.
[v1.0.1]: https://github.com/StarArmyDev/starmusic/compare/v1.0.0...v1.0.1
[v1.0.2]: https://github.com/StarArmyDev/starmusic/compare/v1.0.1...v1.0.2
[v2.0.0]: https://github.com/StarArmyDev/starmusic/compare/v1.0.2...v2.0.0
[v2.0.1]: https://github.com/StarArmyDev/starmusic/compare/v2.0.0...v2.0.1
10 changes: 5 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -296,8 +296,8 @@ const prefix = '!';
const client = new Discord.Client();
client.commands = new Discord.Collection();

// Aquí iniciamos el módulo.
var music = new StarMusic({
// Aquí iniciamos el módulo y lo pasaremos por el cliente.
client.music = new StarMusic({
youtubeKey: 'ApiKey',
djRoles: ['IDRol', 'IDRol2'],
embedColor: 'BLUE'
Expand Down Expand Up @@ -327,7 +327,7 @@ client.on('messages', (message) => {

if (command == 'play') {
// Obtenemos nuestro comando y ejecutamos su función execute.
client.commands.get('play').execute(message, args);
client.commands.get('play').execute(client, message, args);
}
});

Expand All @@ -340,8 +340,8 @@ En la misma carpeta del bot, cree una nueva carpeta y asígnele un nombre como `
module.exports = {
name: 'play',
description: 'Reproduce una canción',
execute(message, args) {
music.play(message, args.join(' '));
execute(client, message, args) {
client.music.play(message, args.join(' '));
}
};
```
Expand Down
4 changes: 2 additions & 2 deletions lib/Music.js
Original file line number Diff line number Diff line change
Expand Up @@ -184,10 +184,10 @@ class Music {
throw `Error interno inesperado: ${error.stack}`;
});
dispatcher.on('error', (error) => {
new Error(`error interno inesperado: ${error.stack}`);
if (message && message.channel)
message.channel.send(this.notaMsg('fail', 'Algo salió mal al tocar música. Volviendo a intentar...'));
this.playSong(message, servidores);
throw `Error interno inesperado: ${error.stack}`;
});
dispatcher.on('finish', () => {
setTimeout(async () => {
Expand Down Expand Up @@ -361,7 +361,7 @@ class Music {
if (!video.duration || !video.datePublished) {
const result = await this._youtube.searchVideos(video.id, 1);
if (result.results[0]) {
video.duration = result.results[0].seconds;
video.duration = result.results[0].minutes * 60 + result.results[0].seconds;
video.datePublished = result.results[0].datePublished;
}
}
Expand Down
8 changes: 4 additions & 4 deletions lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@ const libs_1 = require("./libs");
const Music_1 = __importDefault(require("./Music"));
class StarMusic extends Music_1.default {
play(message, search) {
if (!message.guild)
if (!message.guild || !message.member)
message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));
else if (!message.member.voice.channel)
message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz.'));
else if (!search)
message.channel.send(this.notaMsg('fail', '¡No has colocado nada que buscar!'));
else if (message.member && this._just_dj && (this.isDj(message.member) || this.isAdmin(message.member)))
else if (this._just_dj && (this.isDj(message.member) || this.isAdmin(message.member)))
message.channel.send(this.notaMsg('fail', 'No tienes permitido reproducír música ya que no cuentas con el rol correspondiente.'));
else {
let servidores = this._guilds.get(message.guild.id);
Expand Down Expand Up @@ -112,9 +112,9 @@ class StarMusic extends Music_1.default {
}
}
search(message, search) {
if (!message.guild)
if (!message.guild || !message.member)
message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));
if (!message.member.voice.channel)
else if (!message.member.voice.channel)
message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz'));
else if (!search)
message.channel.send(this.notaMsg('fail', 'No especificaste algo qué buscar'));
Expand Down
4 changes: 2 additions & 2 deletions src/Music.ts
Original file line number Diff line number Diff line change
Expand Up @@ -406,9 +406,9 @@ export default abstract class Music {
});

dispatcher.on('error', (error: Error) => {
new Error(`error interno inesperado: ${error.stack}`);
if (message && message.channel) message.channel.send(this.notaMsg('fail', 'Algo salió mal al tocar música. Volviendo a intentar...'));
this.playSong(message, servidores);
throw `Error interno inesperado: ${error.stack}`;
});

dispatcher.on('finish', () => {
Expand Down Expand Up @@ -601,7 +601,7 @@ export default abstract class Music {
if (!video.duration || !video.datePublished) {
const result = await this._youtube.searchVideos(video.id, 1);
if (result.results[0]) {
video.duration = result.results[0].seconds;
video.duration = result.results[0].minutes * 60 + result.results[0].seconds;
video.datePublished = result.results[0].datePublished;
}
}
Expand Down
13 changes: 6 additions & 7 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ export default class StarMusic extends Music {
// Inicio

play(message: Message, search: string): void {
if (!message.guild) message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));
else if (!message.member!.voice.channel) message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz.'));
if (!message.guild || !message.member) message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));
else if (!message.member.voice.channel) message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz.'));
else if (!search) message.channel.send(this.notaMsg('fail', '¡No has colocado nada que buscar!'));
else if (message.member && this._just_dj && (this.isDj(message.member) || this.isAdmin(message.member)))
else if (this._just_dj && (this.isDj(message.member) || this.isAdmin(message.member)))
message.channel.send(this.notaMsg('fail', 'No tienes permitido reproducír música ya que no cuentas con el rol correspondiente.'));
else {
let servidores = this._guilds.get(message.guild.id);
Expand Down Expand Up @@ -116,9 +116,8 @@ export default class StarMusic extends Music {
}

search(message: Message, search: string): void {
if (!message.guild) message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));

if (!message.member!.voice.channel) message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz'));
if (!message.guild || !message.member) message.channel.send(this.notaMsg('fail', 'No estas en un servidor.'));
else if (!message.member.voice.channel) message.channel.send(this.notaMsg('fail', 'No estas en un canal de voz'));
else if (!search) message.channel.send(this.notaMsg('fail', 'No especificaste algo qué buscar'));
else {
let servidores = this._guilds.get(message.guild!.id);
Expand All @@ -131,7 +130,7 @@ export default class StarMusic extends Music {
isRadio: false
})
.get(message.guild.id);
if (this._just_dj && !this.isDj(message.member!) && !this.isAdmin(message.member!))
if (this._just_dj && !this.isDj(message.member) && !this.isAdmin(message.member))
message.channel.send(this.notaMsg('fail', 'No tienes permitido reproducír música ya que no cuentas con el rol correspondiente.'));
else if (servidores!.songs.length >= this._max_tail && this._max_tail > 0) message.channel.send(this.notaMsg('fail', 'Tamaño máximo de cola alcanzado!'));
else {
Expand Down

0 comments on commit 5138339

Please sign in to comment.