verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 67 Releases Wiki Activity

5066-vehicleSorted #1467

Merged
carlossa merged 23 commits from 5066-vehicleSorted into dev 2023-07-18 15:21:57 +00:00
Conversation 0 Commits 23 Files Changed 5 +10 -14
3 changed files with 10 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit eff46c1cec - Show all commits

2
e2e/paths/08-route/02_basic_data.spec.js
View File

@ -24,7 +24,7 @@ describe('Route basic Data path', () => {
const form = 'vn-route-basic-data form';
const values = {
worker: 'adminBossNick',
vehicle: '1111-IMK - Warehouse One',
vehicle: '1111-IMK',
created: nextMonth,
kmStart: 1,
kmEnd: 2,

3
modules/route/front/basic-data/index.html
View File

@ -28,7 +28,8 @@
data="$ctrl.vehicles"
jgallego commented 2023-05-31 05:34:21 +00:00
Outdated
Review
Copy Link

desde back no has de pasar el concat, al autocomplete busca exemples en el que el autocomplete es genera ell el concat

desde back no has de pasar el concat, al autocomplete busca exemples en el que el autocomplete es genera ell el concat
show-field="numberPlate"
value-field="id"
order="false">
order="false"
vn-name="vehicle">
<tpl-item>{{::numberPlate}} - {{::name}}</tpl-item>
</vn-autocomplete>
</vn-horizontal>

19
modules/route/front/basic-data/index.js
View File

@ -2,23 +2,18 @@ import ngModule from '../module';
import Section from 'salix/components/section';
class Controller extends Section {
$onInit() {
this.$http.post(`Vehicles/sorted`, {warehouseFk: this.vnConfig.warehouseFk})
.then(res => {
this.vehicles = res.data;
});
}
onSubmit() {
juan commented 2023-06-20 12:03:58 +00:00
Outdated
Review
Copy Link

En lugar de hacer la petición, utilizar el servicio vnConfig.

En lugar de hacer la petición, utilizar el servicio `vnConfig`.
this.$.watcher.submit().then(() =>
this.card.reload()
);
}
constructor($element, $) {
super($element, $);
this.$http.get(`UserConfigs/getUserConfig`)
.then(res => {
if (res.data) {
this.$http.post(`Vehicles/sorted`, {warehouseFk: res.data.warehouseFk})
.then(res => {
this.vehicles = res.data;
});
}
});
}
}
juan commented 2023-05-29 07:14:39 +00:00
Outdated
Review
Copy Link

Para que se compruebe si res y res.data está definido?

Para que se compruebe si `res` y `res.data` está definido?
carlossa commented 2023-05-29 10:43:04 +00:00
Outdated
Review
Copy Link

Se utiliza para comprobar que res.data lleva la información, ya que si no da error al buscarlo

Se utiliza para comprobar que res.data lleva la información, ya que si no da error al buscarlo
juan commented 2023-06-06 19:48:46 +00:00
Outdated
Review
Copy Link

En que casos res no esta definido?

En que casos `res` no esta definido?
carlossa commented 2023-06-07 06:32:07 +00:00
Outdated
Review
Copy Link

Lo estaba haciendo como una comprobación de que también tenia res, pero he pensado que directamente comprobando res.data ya serviria ya que es el que lleva la información

Lo estaba haciendo como una comprobación de que también tenia res, pero he pensado que directamente comprobando res.data ya serviria ya que es el que lleva la información
carlossa commented 2023-06-07 06:35:23 +00:00
Outdated
Review
Copy Link

Porque si no habia res, res.data me daria error porque intenta hacer .data de undefined, pero como res esta definido, no hace falta la comprobación de si existe res

Porque si no habia res, res.data me daria error porque intenta hacer .data de undefined, pero como res esta definido, no hace falta la comprobación de si existe res
juan commented 2023-06-20 12:04:41 +00:00
Outdated
Review
Copy Link

El if (res.data) no es necesario y no funcionará correctamente si no hay resultados.

El `if (res.data)` no es necesario y no funcionará correctamente si no hay resultados.
ngModule.vnComponent('vnRouteBasicData', {